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

refactor: fix SA1208 #218

Merged
merged 3 commits into from
Aug 24, 2022
Merged

refactor: fix SA1208 #218

merged 3 commits into from
Aug 24, 2022

Conversation

JamieMagee
Copy link
Member

Related to #202

Using directive for '...' should appear before directive for '...'
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1208.md

Related to #202

Using directive for '...' should appear before directive for '...'
https://github.com/DotNetAnalyzers/StyleCopAnalyzers/blob/master/documentation/SA1208.md
@JamieMagee JamieMagee requested a review from amitla1 August 23, 2022 21:55
@JamieMagee JamieMagee requested a review from a team as a code owner August 23, 2022 21:55
@JamieMagee JamieMagee enabled auto-merge (squash) August 23, 2022 21:57
@github-actions
Copy link

👋 Hi! It looks like you modified some files in the Detectors folder.
You may need to bump the detector versions if any of the following scenarios apply:

  • The detector detects more or fewer components than before
  • The detector generates different parent/child graph relationships than before
  • The detector generates different devDependencies values than before

If none of the above scenarios apply, feel free to ignore this comment 🙂

@JamieMagee JamieMagee merged commit 644512a into main Aug 24, 2022
@JamieMagee JamieMagee deleted the users/jamagee/SA1208 branch August 24, 2022 18:42
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.

2 participants