Skip to content

Fixing code fixes, part 6 - #15773

Merged
psfinaki merged 2 commits into
dotnet:mainfrom
psfinaki:codefixes-23
Aug 11, 2023
Merged

psfinaki merged 2 commits into
dotnet:mainfrom
psfinaki:codefixes-23

Conversation

@psfinaki

@psfinaki psfinaki commented Aug 9, 2023 •

Copy link
Copy Markdown
Contributor

This PR addresses last remaining bulk code fixes - from analyzers.

As usual:

  • adding tests
  • moving to cancellables
  • speeding things up

The test framework is extended to handle IDE diagnostics.

@psfinaki
psfinaki requested a review from a team as a code owner August 9, 2023 13:22
Comment thread vsintegration/src/FSharp.Editor/CodeFixes/RemoveUnusedOpens.fs Outdated
@psfinaki
psfinaki enabled auto-merge (squash) August 11, 2023 09:42
@psfinaki
psfinaki merged commit 3e58e14 into dotnet:main Aug 11, 2023
@psfinaki
psfinaki deleted the codefixes-23 branch August 11, 2023 10:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants