Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

gh-113336: Remove the 'version' directive from Argument Clinic #113341

Merged

Conversation

erlend-aasland
Copy link
Contributor

@erlend-aasland erlend-aasland commented Dec 20, 2023

The 'version' directive was introduced with gh-63929 in Nov 2013. It has
not been in use in the CPython code base, and the 'version' variable has
never been bumped.

The 'version' directive was introduced with pythongh-63929 in Nov 2013. It has
not been in use in the CPython code base, and the 'version' variable has
never been bumped.
Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

LGTM, though I'm a tiny bit worried about Chesterton's fence here. I'd love it if @serhiy-storchaka could also take a look!

@erlend-aasland
Copy link
Contributor Author

Thanks for the reviews, Alex and Serhiy.

@erlend-aasland erlend-aasland merged commit fae096c into python:main Dec 21, 2023
37 checks passed
@erlend-aasland erlend-aasland deleted the clinic/remove-version-directive branch December 21, 2023 12:10
ryan-duve pushed a commit to ryan-duve/cpython that referenced this pull request Dec 26, 2023
…ython#113341)

The 'version' directive was introduced with pythongh-63929 in Nov 2013. It has
not been in use in the CPython code base, and the 'version' variable has
never been bumped.
kulikjak pushed a commit to kulikjak/cpython that referenced this pull request Jan 22, 2024
…ython#113341)

The 'version' directive was introduced with pythongh-63929 in Nov 2013. It has
not been in use in the CPython code base, and the 'version' variable has
never been bumped.
aisk pushed a commit to aisk/cpython that referenced this pull request Feb 11, 2024
…ython#113341)

The 'version' directive was introduced with pythongh-63929 in Nov 2013. It has
not been in use in the CPython code base, and the 'version' variable has
never been bumped.
Glyphack pushed a commit to Glyphack/cpython that referenced this pull request Sep 2, 2024
…ython#113341)

The 'version' directive was introduced with pythongh-63929 in Nov 2013. It has
not been in use in the CPython code base, and the 'version' variable has
never been bumped.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants