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

[RC3] Code Fixes do not work #2329

Closed
Tracked by #15408
vasily-kirichenko opened this issue Jan 27, 2017 · 6 comments · Fixed by dotnet/roslyn#16799
Closed
Tracked by #15408

[RC3] Code Fixes do not work #2329

vasily-kirichenko opened this issue Jan 27, 2017 · 6 comments · Fixed by dotnet/roslyn#16799
Labels
Area-LangService-API Area-LangService-CodeFixes Code fixes associated with diagnostics Impact-High (Internal MS Team use only) Describes an issue with extreme impact on existing code.
Milestone

Comments

@vasily-kirichenko
Copy link
Contributor

I attached to a VS instance and don't see any exceptions, like missing methods, etc. Absolutely no idea what's happening. Code Fixes work OK in C# editor.

@cartermp cartermp added Area-LangService-API Impact-High (Internal MS Team use only) Describes an issue with extreme impact on existing code. labels Jan 27, 2017
@cartermp cartermp added this to the VS 2017 Updates milestone Jan 27, 2017
@cartermp
Copy link
Contributor

Yup, I think the bit that ultimately connects to the "initiate a code fix suggestion" is disabled somehow...you can see the Simplifier Analyzer actually working by dimming/graying tokens which can be simplified.

@vasily-kirichenko
Copy link
Contributor Author

Maybe update VS packages? There's 15.0.26105-RC3 version.

@saul
Copy link
Contributor

saul commented Jan 27, 2017

I can't find it for the life of me, but there was a PR on the Roslyn project that disabled Code Fixes for F# - I brought this up at the time it was merged but it looks there was no reply.

I believe the PR disabled "Go to implementation" if there wasn't an implementation of it in the language service. I'll keep trying to find the PR...

@saul
Copy link
Contributor

saul commented Jan 27, 2017

See dotnet/roslyn#15961

We need to add the FSharp content type to src/EditorFeatures/Core/Implementation/Suggestions/SuggestedActionsSourceProvider.cs in Roslyn

@cartermp
Copy link
Contributor

Thanks, @saul! Looks like that change it getting taken to our shiproom, so if it's approved then I'll change the milestone here and close this out.

@saul
Copy link
Contributor

saul commented Jan 27, 2017

@cartermp it's been approved and merged 👍

@cartermp cartermp modified the milestones: VS 2017 Updates, VS 2017 RTM Jun 9, 2018
@psfinaki psfinaki added the Area-LangService-CodeFixes Code fixes associated with diagnostics label Jun 15, 2023
@psfinaki psfinaki mentioned this issue Jun 15, 2023
85 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-LangService-API Area-LangService-CodeFixes Code fixes associated with diagnostics Impact-High (Internal MS Team use only) Describes an issue with extreme impact on existing code.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants