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

[Feature Request] Better static analyzers in MSAL .NET #2921

Closed
3 of 5 tasks
bgavrilMS opened this issue Sep 30, 2021 · 0 comments
Closed
3 of 5 tasks

[Feature Request] Better static analyzers in MSAL .NET #2921

bgavrilMS opened this issue Sep 30, 2021 · 0 comments
Assignees
@bgavrilMS
Labels
Feature Request internal

Comments

@bgavrilMS
Copy link
Member

bgavrilMS commented Sep 30, 2021 •
edited by trwalke
Loading

Our current static analyzers do not catch enough issues.

  • Investigate upgrading to the latest version of C# to take advantage of all language features. Not all features will be available on .NET fwk, but most should be.

  • Upgrade the current async analyzer, as it is deprecated

  • Review .NET static analyzers rules and see the impact of enabling the recommended rules

  • Add some rules which do not imply lots of work

  • Discuss with the team rules that imply lots of work
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bgavrilMS bgavrilMS

Labels
Feature Request internal
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@bgavrilMS @trwalke @pmaytak