-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
parser, checker, pref: allow getting notified about unused function params #22879
Conversation
It would be great if Or, that there was a command, something like |
Exclude pub fns Exclude receiver param
With this I see some situations where a format on save would replace the param name that is not in use yet but I want to use a few moments later. |
76a261c
to
0f70595
Compare
So it would be better to have a special command for automatic fixes: |
That's a great idea, I've always wanted something like that. Can fix |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work.
The PR should implement #22875 as desired.
Example:
NOTE: the v-analyzer hints won't work until this is on by default. For now, when compiling with
-check-unused-fn-args
you'll get:Huly®: V_0.6-21322