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

Use Operations instead of SyntaxNodes for analysis #177

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

FantasyTeddy
Copy link

The main change in this PR is to use Operations instead of SyntaxNodes to perform the code analysis.

Additionally, the following smaller changes have been introduced:

  • Add a test case for non-related fields that should not trigger the analyzer.
  • Use Directory.Delete instead of the non-existing Directory.Remove method in the "WithOutFileSystem.txt" test for Directory.
  • Remove the "StaticInvocation.txt" test for DirectoryInfo, because no real static members exist for DirectoryInfo.
  • Use the .New() method for all code fixes instead of their previous counterpart.
  • Remove the ConfigurationBuilder in the Directory code fix test, because this does not compile.
  • Use Path.GetFullPath instead of Path.Exists, because the latter does not exist in .NET 6 (which the project currently still uses).

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.

None yet

1 participant