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.

Sign up for GitHub

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

Jump to bottom

gh-104146: Purge CConverter.parse_by_reference from Argument Clinic #119162

Closed

erlend-aasland wants to merge 1 commit into python:main from erlend-aasland:clinic/parse-by-ref

Contributor

erlend-aasland commented May 18, 2024 •

edited by bedevere-app bot

Loading

'parse_by_reference' was always set to True, and it has always been like that since it's introduction. I think we can remove it.

Issue: Remove dead code from clinic.py #104146


          pythongh-104146: Purge CConverter.parse_by_reference from Argument Cl…

…inic

bedevere-app bot added the awaiting core review label

bedevere-app bot mentioned this pull request

Remove dead code from clinic.py #104146

Closed

erlend-aasland added the skip news label

Contributor Author

erlend-aasland commented May 18, 2024

On other thought, let's leave it; a custom converter may use it.

erlend-aasland closed this

erlend-aasland deleted the clinic/parse-by-ref branch

May 18, 2024 22:03

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting core review skip news