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

PSSCriptAnalyzer Needs Rule Sets #383

Open
Jaykul opened this issue Nov 28, 2015 · 6 comments
Open

PSSCriptAnalyzer Needs Rule Sets #383

Jaykul opened this issue Nov 28, 2015 · 6 comments

Comments

@Jaykul
Copy link
Contributor

Jaykul commented Nov 28, 2015

2015-11-26_20-20-19

We need rule sets for minimum safety, basic design, recommended design, etc. When you start bringing in rules like mandating named parameters or placement of curly braces, you are way past the point where you should have started collecting them into sets.

@kilasuit
Copy link
Contributor

Agree - and there was a discussion on what these should be named - previously was Profiles (agreed bad name and therefore changed) but changed to configuration - which I very much disagree with due to configuration being a keyword and I would be more suggestive to rename this (again) to RuleSet

This is more generic and would be more better for discoverability with an additional about topic being packed with it

@rkeithhill
Copy link
Contributor

Yeah given that the tool has "rules", ruleset is a better name than profile (which I think I might have proposed??). And I would like to see the "default" or "minimum" ruleset be something like the C# "Microsoft Basic Correctness Rules" or "Microsoft Managed Minimum Rules" ruleset.

PSScriptAnalyzer's default ruleset should contain only the rules that have a very high probably of indicating a real problem. I believe this will give the best "out of the box" experience and if users want more, well then the team should have more rulesets that the user can choose from.

@kilasuit
Copy link
Contributor

kilasuit commented Dec 1, 2015

This really needs to be called RuleSet as not to add further confusion to new user with core Keywords or Default variables as opposed to the current implementation which replaced profile with configuration

See
#165
&
#370

@joeyaiello
Copy link
Contributor

Agreed with @kilasuit here. I apologize for missing the conversation on Profile vs. Configuration as I would have been in favor of leaving the original parameter name (i.e. Profile), given the fact it's a breaking change. However, since we haven't properly documented or implemented any yet, I would also be open to a breaking change on the parameter name, but Configuration is absolutely confusing as a PowerShell keyword.

Thoughts?

@KirkMunro
Copy link
Contributor

I like RuleSet. If it looks like a duck and acts like a duck...

@joeyaiello
Copy link
Contributor

Cool, we'll raise it on the call today and see if anyone has any objections.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants