Skip to content

Use a minimal set of Script Analyzer rules #70

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

Merged
merged 2 commits into from
Dec 14, 2015

Conversation

daviwil
Copy link
Contributor

@daviwil daviwil commented Dec 11, 2015

This change greatly reduces the set of Script Analyzer rules that are used
by default in the analysis service. In the future, this list of rules
will be fully configurable by the user. This change is meant to simplify
the ruleset in the meantime.

@daviwil
Copy link
Contributor Author

daviwil commented Dec 11, 2015

cc @Jaykul @rkeithhill

@daviwil
Copy link
Contributor Author

daviwil commented Dec 12, 2015

Keith recommends adding PSUseDeclaredVarsMoreThanAssigments, any disagreement there? I'll merge this PR sometime tomorrow if nobody has objections to the list of rules.

@daviwil
Copy link
Contributor Author

daviwil commented Dec 13, 2015

Need to take out PSUseSingularNouns before merging.

This change greatly reduces the set of Script Analyzer rules that are used
by default in the analysis service.  In the future, this list of rules
will be fully configurable by the user.  This change is meant to simplify
the ruleset in the meantime.
@daviwil daviwil force-pushed the daviwil/minimal-analysis-rules branch from f9d8015 to ba69936 Compare December 14, 2015 16:11
daviwil added a commit that referenced this pull request Dec 14, 2015
Use a minimal set of Script Analyzer rules
@daviwil daviwil merged commit 4397b4a into master Dec 14, 2015
@daviwil daviwil deleted the daviwil/minimal-analysis-rules branch December 14, 2015 16:14
TylerLeonhardt pushed a commit to TylerLeonhardt/PowerShellEditorServices that referenced this pull request Feb 26, 2019
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