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.

Sign up for GitHub

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

Jump to bottom

Fix MSTEST0022 code fix formatting #3790

Closed
engyebrahim opened this issue Sep 9, 2024 · 1 comment · Fixed by #4000
Closed

Fix MSTEST0022 code fix formatting #3790

engyebrahim opened this issue Sep 9, 2024 · 1 comment · Fixed by #4000
Assignees
@Youssef1313
Labels
Area: Analyzers In-PR sprint Type: Bug
Milestone
MSTest 3.7 / Plat...

Comments

@engyebrahim
Copy link
Member

engyebrahim commented Sep 9, 2024 •
edited by pavelhorak
Loading

related to: #3770 (comment)

AB#2296027
@engyebrahim
Copy link
Member Author

and fix that line: https://github.com/microsoft/testfx/blob/main/src/Analyzers/MSTest.Analyzers.CodeFixes/PreferTestCleanupOverDisposeFixer.cs#L52

@engyebrahim engyebrahim self-assigned this Sep 10, 2024
@engyebrahim engyebrahim added sprint and removed sprint labels Sep 10, 2024
@Evangelink Evangelink added this to the MSTest 3.7 / Platform 1.5 milestone Oct 31, 2024
@Evangelink Evangelink assigned Youssef1313 and unassigned engyebrahim Nov 4, 2024
@Youssef1313 Youssef1313 mentioned this issue Nov 4, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Youssef1313 Youssef1313

Labels
Area: Analyzers In-PR sprint Type: Bug
Projects
None yet
Milestone
MSTest 3.7 / Platform 1.5
Development

Successfully merging a pull request may close this issue.

Handle trivia properly in PreferTestCleanupOverDisposeFixer Youssef1313/testfx
3 participants
@Evangelink @engyebrahim @Youssef1313