Update SA1649CodeFixProvider to fix problems related to linked files #3862
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
bjornhellander
force-pushed
the
feature/sa1649-codefix-problems
branch
from
4f91b3e
to
1d22cdb
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #3862 +/- ##
=======================================
Coverage 97.45% 97.46%
=======================================
Files 926 926
Lines 110237 110292 +55
Branches 3315 3316 +1
=======================================
+ Hits 107433 107491 +58
+ Misses 1837 1834 -3
Partials 967 967 |
…ts which reference the same file before adding them there (to not end up with two identical files in those projects) and also to re-add files with correct path (to not get a copy where there was a link before) DotNetAnalyzers#1693 DotNetAnalyzers#3866
bjornhellander
force-pushed
the
feature/sa1649-codefix-problems
branch
from
1d22cdb
to
282c5e1
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1693 and #3866.
Unsure how to unit test this properly, but I have also used the code fix "live" in this repository with good success:
https://github.com/bjornhellander/SA1649CodeFixProblem
Shared Projects still doesn't work, with exceptions being thrown and the fix messing up the projects, but it does not seem to behave worse than before. (The built-in rename refactoring MoveTypeCodeRefactoringProvider also throws exception on this, but handles files and projects better.) But normal linked files between projects seem to be handled correctly with these changes.
I have started looking at the new APIs to rename files, but since these issues seems to be solvable without them I didn't add those part in this PR.