Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Sep 23, 2025

This PR completes the pytest version specification updates started in #233 by updating the remaining GitHub Actions workflow files that still used the old version constraints.

Problem

While PR #233 successfully updated test-functions.yml and verify-module.yml to use modern pytest versions that resolve serialization issues, 4 other workflow files were left with the old version specifications:

  • .github/workflows/run-examples.yml
  • .github/workflows/test-functions-python313.yml
  • .github/workflows/test-pip.yml
  • .github/workflows/measure-coverage.yml

This inconsistency could lead to different pytest behaviors across workflows and potential serialization failures in parallel test execution.

Solution

Updated all remaining workflow files to use the same pytest version specifications established in #233:

  • pytest>=7.4.0,<8.0.0pytest>=8.4.0
  • pytest-xdist>=3.3.0,<4.0.0pytest-xdist>=3.8.0
  • pytest-rerunfailures>=11.1,<12.0pytest-rerunfailures>=16.0

Also updated the comments to consistently reference "parallel execution" instead of "serialization issues" to match the style from #233.

Impact

  • Consistency: All 6 GitHub Actions workflows now use identical pytest dependency versions
  • Reliability: Eliminates potential serialization issues in parallel test execution across all workflows
  • Performance: Ensures all workflows benefit from the improved pytest stability and parallel execution capabilities
  • Maintainability: Prevents future confusion about which pytest versions should be used

Fixes #235.


💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.

Copilot AI changed the title [WIP] pytest submodule version specification inconsistency Update remaining workflow files with consistent pytest version specifications Sep 23, 2025
Copilot AI requested a review from toruseo September 23, 2025 03:34
@toruseo toruseo marked this pull request as ready for review September 23, 2025 07:48
@toruseo toruseo enabled auto-merge (squash) September 23, 2025 07:48
@toruseo toruseo merged commit 539d00f into main Sep 23, 2025
20 of 23 checks passed
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.

pytest submodule version specification inconsistency

2 participants