Skip to content

Conversation

Tpt
Copy link
Contributor

@Tpt Tpt commented Apr 25, 2025

Positional-only arguments are not allowed in Python 3.7

@Tpt Tpt added the CI-skip-changelog Skip checking changelog entry label Apr 25, 2025
@Tpt Tpt requested a review from ngoldbaum April 25, 2025 19:14
@Tpt
Copy link
Contributor Author

Tpt commented Apr 25, 2025

See #5025 (comment)

No changelog, the test has been introduced in an unreleased change

@ngoldbaum
Copy link
Contributor

Closing and reopening to trigger the CI-build-full tests, hopefully.

@ngoldbaum ngoldbaum closed this Apr 25, 2025
@ngoldbaum ngoldbaum reopened this Apr 25, 2025
@Tpt Tpt changed the title Disable positional only test in Python 3.7 Fixes pytests tests when using Python 3.7 Apr 25, 2025
@ngoldbaum ngoldbaum enabled auto-merge April 25, 2025 20:48
@ngoldbaum
Copy link
Contributor

Seems to have fixed the 3.7 build, thanks! Added to the queue.

@ngoldbaum ngoldbaum added this pull request to the merge queue Apr 25, 2025
Merged via the queue into PyO3:main with commit 61f43e8 Apr 25, 2025
90 checks passed
newcomertv pushed a commit to newcomertv/pyo3 that referenced this pull request Apr 28, 2025
* Disable positional only test in Python 3.7

* Update pytests/tests/test_pyfunctions.py

Co-authored-by: Nathan Goldbaum <[email protected]>

* fix ruff

---------

Co-authored-by: Nathan Goldbaum <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI-build-full CI-skip-changelog Skip checking changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants