Skip to content

Conversation

@alex
Copy link
Contributor

@alex alex commented Aug 18, 2025

these have all been deprecated in CPython since 3.12, will be removed in 3.14, and we've deprecaed them for more than 2 years.

also in the latest rust nightly, #[deprecated] doesn't work on statics and emits warnings

@alex alex force-pushed the remove-deprecated-statics branch 3 times, most recently from f2eac13 to 3c5808b Compare August 18, 2025 11:48
these have all been deprecated in CPython since 3.12, will be removed in 3.14, and we've deprecaed them for more than 2 years.

also in the latest rust nightly, `#[deprecated]` doesn't work on statics and emits warnings
@alex alex force-pushed the remove-deprecated-statics branch from 3c5808b to db41e02 Compare August 18, 2025 11:50
Co-authored-by: Lily Acorn <code@lilyf.org>
Copy link
Member

@davidhewitt davidhewitt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm ok with doing this as long as users on older Python versions have alternatives?

@alex
Copy link
Contributor Author

alex commented Aug 20, 2025

I'm not sure there is a great alternative, I put this up because of the warnings we were getting that deprecated didn't work on statics, I think those are otherwise resolved now?

@Icxolu
Copy link
Contributor

Icxolu commented Aug 20, 2025

Yes, that was a nightly regression fixed by rust-lang/rust#145485

@alex
Copy link
Contributor Author

alex commented Aug 20, 2025 via email

@davidhewitt
Copy link
Member

👍 seems reasonable to play it safe, maybe when we get to dropping Python 11 support we can drop these at the same time (presumably Python 3.12 had suitable alternatives when they were deprecated).

@alex alex deleted the remove-deprecated-statics branch August 20, 2025 18:49
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.

4 participants