Conversation
The `*` import was shadowing the name of the pyclass. The PyMethods trait was only defined for the multiple-pymethods feature, but it cannot be conditionally imported inside the macro. I just defined the trait unconditionally. Fixes #4792.
* fix: update type import cfg * add: add newsfragments
* Fix unsoundness in `PyBytesWriter::write_vectored` * Add testcase * Update safety comment on `PyBytesWriter::set_len` * Apply suggestion Co-authored-by: Icxolu <10486322+Icxolu@users.noreply.github.com> --------- Co-authored-by: Icxolu <10486322+Icxolu@users.noreply.github.com>
Merging this PR will degrade performance by 21.15%
Performance Changes
Comparing Footnotes |
* update UI tests for Rust 1.94 * ci: debug windows introspection failure * check path for failed object * ensure file path for `test-introspection` is correct
* fix deadlock when initializing enum via `into_pyobject()` * newsfragment * fix `unreachable_code` failure
|
Think we should backport #5917 as well? |
|
I actually deliberately skipped #5917 because it could break existing compiles. Thought it's probably best to defer that to 0.29.0, which probably means we should aim to ship that before too long. I'll try to get this shipped later today, sorry for the delay, ran out of energy yesterday evening and dropped the ball on a few different things. |
52e762c to
a11c6f5
Compare
|
(force pushed to update release date to today, will merge after CI and release) |
a11c6f5 to
492eba7
Compare
|
I fell asleep before CI finished running yesterday - need to get more optimizations landed 😅 |
Backports for an upcoming 0.28.3 release.
Will wait until #5928 is resolved (to cherry-pick it here) and then I think we're ready to ship this set of fixes.