Skip to content

delete the ability to access a function parameter name from reflection #6534

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

Closed
andrewrk opened this issue Oct 4, 2020 · 3 comments
Closed
Labels
accepted This proposal is planned. proposal This issue suggests modifications. If it also has the "accepted" label then it is planned.
Milestone

Comments

@andrewrk
Copy link
Member

andrewrk commented Oct 4, 2020

Similar reasoning as ability to access doc comments with reflection. Too many ways to make confusing code. We don't want people reading code to have to wonder in the back of their minds if naming a function parameter a certain way would have a meaningful change.

related: #6530

@andrewrk andrewrk added the proposal This issue suggests modifications. If it also has the "accepted" label then it is planned. label Oct 4, 2020
@andrewrk andrewrk added this to the 0.8.0 milestone Oct 4, 2020
@andrewrk andrewrk modified the milestones: 0.8.0, 0.9.0 May 19, 2021
@andrewrk andrewrk modified the milestones: 0.9.0, 0.10.0 Nov 23, 2021
@andrewrk andrewrk modified the milestones: 0.10.0, 0.11.0 Apr 16, 2022
@andrewrk andrewrk modified the milestones: 0.11.0, 0.12.0 Apr 9, 2023
@andrewrk andrewrk modified the milestones: 0.13.0, 0.12.0 Jul 9, 2023
@andrewrk andrewrk added the accepted This proposal is planned. label Feb 9, 2025
@andrewrk andrewrk modified the milestones: 0.14.0, 0.15.0 Feb 9, 2025
@mlugg
Copy link
Member

mlugg commented Feb 10, 2025

Isn't this already impossible? @andrewrk

@nektro
Copy link
Contributor

nektro commented Feb 10, 2025

yes since #10753

@mlugg
Copy link
Member

mlugg commented Feb 10, 2025

Oh man, I totally forgot we used to have that. Thanks for the reference!

@mlugg mlugg closed this as completed Feb 10, 2025
@mlugg mlugg modified the milestones: 0.15.0, 0.14.0 Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted This proposal is planned. proposal This issue suggests modifications. If it also has the "accepted" label then it is planned.
Projects
None yet
Development

No branches or pull requests

3 participants