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

Record whether a CodeAction is a fix or not #2430

Merged
merged 2 commits into from
Aug 11, 2022

Conversation

333fred
Copy link
Contributor

@333fred 333fred commented Aug 10, 2022

Omnisharp-roslyn side of fixing dotnet/vscode-csharp#5335.

Copy link

@Rekkonnect Rekkonnect left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not too sure about the choice of using tuples of the form (CodeAction, bool) instead of a record like CodeActionInfo(CodeAction Action, bool IsCodeFix) (not the best name I came up with)

@333fred
Copy link
Contributor Author

333fred commented Aug 10, 2022

Tuples are great for these types of intermediate structure scenarios.

…e LSP handler code for this into the main handler, and more accurately classify quickfix actions.
@333fred
Copy link
Contributor Author

333fred commented Aug 11, 2022

@JoeRobich @filipw for review please.

333fred added a commit to 333fred/vscode-csharp that referenced this pull request Aug 11, 2022
Copy link
Member

@JoeRobich JoeRobich left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@333fred 333fred merged commit 80d7f26 into OmniSharp:master Aug 11, 2022
@333fred 333fred deleted the categorize-codeactions branch August 11, 2022 18:39
333fred added a commit to dotnet/vscode-csharp that referenced this pull request Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants