Skip to content

Conversation

@Icxolu
Copy link
Contributor

@Icxolu Icxolu commented Aug 15, 2025

This deprecates the PyObject type alias for Py<PyAny>. This alias is confused easily with pyo3_ffi::PyObject, last time #4434 (comment).

This may fit nicely with all the other renaming cleanups in 0.26, but I'm also fine postponing until 0.27 if we think it's to much.

@Icxolu Icxolu force-pushed the deprecate/pyobject branch from e6b91e2 to 30c1139 Compare August 15, 2025 19:52
@davidhewitt
Copy link
Member

👍 I think we should do this; with known reasons to cause confusion I agree that the alias does more harm than good.

I think that this is probably safe to send in 0.26, then the release continues to carry its theme as "a name cleanup". I am cautious about putting much else at all into 0.27 given how the FromPyObject change might be quite fiddly for downstream, and I don't think there's much point holding this back to 0.28 over 0.26.

Let's add a migration guide entry?

@Icxolu Icxolu force-pushed the deprecate/pyobject branch from 30c1139 to ed14f13 Compare August 15, 2025 21:08
@Icxolu
Copy link
Contributor Author

Icxolu commented Aug 15, 2025

Great! I've added the migration entry

@Icxolu Icxolu enabled auto-merge August 15, 2025 21:19
@Icxolu Icxolu force-pushed the deprecate/pyobject branch from 6e5a8de to 5994dca Compare August 16, 2025 09:13
@Icxolu Icxolu added this pull request to the merge queue Aug 16, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 16, 2025
@LilyFirefly LilyFirefly added this pull request to the merge queue Aug 16, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Aug 16, 2025
@Icxolu Icxolu added this pull request to the merge queue Aug 16, 2025
Merged via the queue into PyO3:main with commit 4bef324 Aug 16, 2025
39 of 41 checks passed
@Icxolu Icxolu deleted the deprecate/pyobject branch August 16, 2025 23:50
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.

3 participants