| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
I'm a bit unsure where to really put the name of that constant. I do want users to be able to use the constant itself but I don't want them to change it. Should I add an additional note saying that the value should not be changed? |
Sorry, something went wrong.
There was a problem hiding this comment.
I'm a bit unsure where to really put the name of that constant. I do want users to be able to use the constant itself but I don't want them to change it. Should I add an additional note saying that the value should not be changed?
that's probably overthinking it, we have all sorts of constants in Python modules that if changed will cause bad behavior - I don't think any special callout is necessary.
if we wanted to prevent it from being changed that'd require getting too creative with the module dict or using a C extension module instead. We rejected PEP-726 that could've offered an ability because the actual need was so rare.
Sorry, something went wrong.
Co-authored-by: Gregory P. Smith <68491+gpshead@users.noreply.github.com>
|
Thanks @picnixz for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
Sorry, something went wrong.
pythonGH-144568) (cherry picked from commit 9349972) Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
pythonGH-144568) (cherry picked from commit 9349972) Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
|
GH-144579 is a backport of this pull request to the 3.14 branch. |
Sorry, something went wrong.
|
GH-144580 is a backport of this pull request to the 3.13 branch. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
📚 Documentation preview 📚: https://cpython-previews--144568.org.readthedocs.build/