-
Notifications
You must be signed in to change notification settings - Fork 5
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
Code Action Idea: Fix Missing Argument From Context #29
Comments
Resharper has an entire Wizard for this. Basically, these are the resolution options:
Ideally you could use breadth search on all symbols in scope to find valid expressions. Also, if there are multiple possible expressions, pre-select the one that has the best levenshtein distance to parameter name (e.g. parameter name is I think this feature could be a wow-factor. |
Wow, thanks for this awesome writeup! "Introduce parameter" is one of the refactorings I was thinking about :) Just for me to understand your flow better (because the title says "Fix missing argument"), would you primarily need to
|
I would use this for both. If this recorder tool existed, I could show you exactly what I did and in which way automatisation would have helped me! (and how much time I spent on doing it manually) |
This is pretty cool! P42 is currently limited to type information from a single file for performance reasons - I would need to extend the architecture to be able to get type information that's comparable to what the TS service offers. Do you know if there is a way to access the TS type information from other extensions? |
Cursor is on the squiggles.
- Use missing argument from context ->
- Use missing argument from parameter ->
If you could record my code changes, you would see that parameter "wiring" is a task I do very often.
The text was updated successfully, but these errors were encountered: