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

Extend analyzers #259

Merged
merged 6 commits into from
Apr 10, 2024
Merged

Extend analyzers #259

merged 6 commits into from
Apr 10, 2024

Conversation

Corniel
Copy link
Contributor

@Corniel Corniel commented Mar 9, 2024

To further improve the code base the following analyzers have been enabled:

I moved the configuration about the analyzers from the .editorconfig to the .globalconfig file.

@phmonte
Copy link
Owner

phmonte commented Apr 7, 2024

@Corniel I'm ready to merge, If you can resolve the conflicts, if you can't, I can resolve them, just let me know, Thanks.

@Corniel
Copy link
Contributor Author

Corniel commented Apr 8, 2024

Rebase applied.

@phmonte
Copy link
Owner

phmonte commented Apr 10, 2024

@Corniel ,
Thanks, it took me a while because I was reading about DotNetProjectFile.Analyzers and IDisposableAnalyzers

@phmonte phmonte merged commit ffe7787 into phmonte:main Apr 10, 2024
3 checks passed
@Corniel Corniel deleted the global-config branch April 10, 2024 07:44
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