Skip to content

Conversation

@davidhewitt
Copy link
Member

Holding branch for cherry picks for the 0.24.2 release.

davidhewitt and others added 14 commits April 4, 2025 10:49
* Use std::ptr::eq where relevant

* allow(clippy::ptr_eq) in pyo3-ffi

* Add ref for allow(clippy::large_enum_variant)
* ci: change default Python version to 3.13

* fix FFI definition for `PyCodeObject`
* Update noxfile.py

* Update pull_request_template.md
* add

* add

* remove components, not used

* use nox to publish

* Update 5027.packaging.md

* no newsfragment needed for ci change

* add release environment

---------

Co-authored-by: David Hewitt <[email protected]>
* Update building-and-distribution.md

Fix typo

* Update guide/src/building-and-distribution.md
* Unraw complex struct enum field python names

Stray case related to #2395 (probably unnoticed because the functionality is
newer and a bit of an edge case).

The `python_name` of struct-type enum variant fields (and the `__match_args__`
of the variant) do not unraw the names of these fields, which effectively
makes them unusable as the resulting identifiers contain '#' which the Python
lexer interprets as comments. The only reason you'd probably want to do this
anyway is to use reserved identifiers (`type` in my case).

* Add newsfragment for fix

* Fix clippy lints in impl_complex_enum_variant_match_args and callers

No need to pass mutable vec ref anymore

* don't overeagerly collect field_names iterator

* add test case for complex enums containing raw identifiers

* only test raw ident pattern matching on Python 3.10+
@davidhewitt davidhewitt marked this pull request as ready for review April 11, 2025 08:37
@davidhewitt
Copy link
Member Author

I think this is ready to go, will put live on Sunday evening unless I hear reason to delay.

@davidhewitt
Copy link
Member Author

#5065 (comment) may delay this

@davidhewitt
Copy link
Member Author

All good, #5069 demonstrates this is not an issue in the PyO3 library, just the test suite.

@Vrajs16
Copy link
Contributor

Vrajs16 commented Apr 13, 2025

Is it possible to add the bigdecimal feature to this release as well or would that be in a future release?

@davidhewitt
Copy link
Member Author

Sorry for the long delay here. Family commitments again left me no time to push this. I will update the release date now and seek to push this tonight.

Is it possible to add the bigdecimal feature to this release as well or would that be in a future release?

As this is a patch release I will not pick that feature to here; we will release 0.25 in the not too distant future and ship the feature then.

@davidhewitt davidhewitt merged commit 3e6bdce into release-0.24 Apr 21, 2025
44 of 45 checks passed
@davidhewitt davidhewitt deleted the release-0.24.2 branch April 21, 2025 21:17
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.