Add more rules to the static analysis ruleset #1354

Open
opened 2026-01-30 22:23:30 +00:00 by claunia · 3 comments
Owner

Originally created by @adiviness on GitHub (May 24, 2019).

There is a ruleset file for static analysis checks at /src/StaticAnalysis.ruleset but it currently has only two rules that it checks for. More should be added.

Originally created by @adiviness on GitHub (May 24, 2019). There is a ruleset file for static analysis checks at `/src/StaticAnalysis.ruleset` but it currently has only two rules that it checks for. More should be added.
claunia added the Issue-TaskProduct-MetaArea-CodeHealth labels 2026-01-30 22:23:30 +00:00
Author
Owner

@dlong11 commented on GitHub (May 25, 2019):

@adiviness What is the plan here - Is it to identify which rules should be added? Then add all of the passing ones and create issues for the rest?

@dlong11 commented on GitHub (May 25, 2019): @adiviness What is the plan here - Is it to identify which rules should be added? Then add all of the passing ones and create issues for the rest?
Author
Owner

@adiviness commented on GitHub (May 25, 2019):

I guess this particular issue is for the general “more rules should be added” discussion. If someone wants to add a rule, they would make a new issue that specified what rule(s) they’re adding and assign the issue to themself to claim it. I’d like us to have as many checks as possible turned on and I’m sure that we’re not compliant with a bunch of them today.

@adiviness commented on GitHub (May 25, 2019): I guess this particular issue is for the general “more rules should be added” discussion. If someone wants to add a rule, they would make a new issue that specified what rule(s) they’re adding and assign the issue to themself to claim it. I’d like us to have as many checks as possible turned on and I’m sure that we’re not compliant with a bunch of them today.
Author
Owner

@miniksa commented on GitHub (Sep 11, 2019):

#2607 applied full cppwinrt ruleset to 4 projects.

@miniksa commented on GitHub (Sep 11, 2019): #2607 applied full cppwinrt ruleset to 4 projects.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#1354