Skip to content

Fix Python wheel test skipping and dependencies#876

Merged
ColinLeeo merged 1 commit into
developfrom
colin/fix-python-wheel-ci
Jul 23, 2026
Merged

Fix Python wheel test skipping and dependencies#876
ColinLeeo merged 1 commit into
developfrom
colin/fix-python-wheel-ci

Conversation

@ColinLeeo

@ColinLeeo ColinLeeo commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Summary

This PR fixes two issues in the Python wheel build:

  • Make Maven's -DskipTests option correctly skip Python test setup and execution.
  • Use a newer pandas version for Python 3.13 and 3.14 while preserving the existing dependency range for Python 3.9–3.12.

Root Cause

${skipTests:-false} uses shell-style default-value syntax, which is not supported by Maven property interpolation. As a result, -DskipTests was not correctly propagated to the Python test executions.

Additionally, Python 3.13 was still selecting pandas 2.2.2. That version does not provide a compatible Windows CPython 3.13 wheel, causing pip to attempt a source build and fail during the wheel workflow.

Changes

  • Replace ${skipTests:-false} with ${skipTests} in the Python Maven configuration.
  • Select pandas 2.2.2 / <2.3 for Python 3.9–3.12.
  • Select pandas 2.3.3 or newer for Python 3.13 and 3.14.
  • Keep requirements.txt and wheel package metadata consistent.

Validation

  • Verified that the XML and TOML files parse successfully.
  • Verified dependency marker selection for Python 3.9, 3.12, 3.13, and 3.14.
  • Verified through Maven's effective POM that -DskipTests sets both Python test executions to skip=true.
  • Ran git diff --check.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.43%. Comparing base (23787f8) to head (624a24a).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #876   +/-   ##
========================================
  Coverage    60.42%   60.43%           
========================================
  Files          744      744           
  Lines        48963    48963           
  Branches      7771     7771           
========================================
+ Hits         29588    29589    +1     
  Misses       17965    17965           
+ Partials      1410     1409    -1     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ColinLeeo
ColinLeeo merged commit 608955c into develop Jul 23, 2026
29 checks passed
@ColinLeeo
ColinLeeo deleted the colin/fix-python-wheel-ci branch July 23, 2026 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants