PowerShell / PowerShell/PSScriptAnalyzer
Compiler warning in UseCompatibleCmdlets class due unfinished feature with # if DEBUG pragma -> consider removal to allow us to use the /warnaserror compiler option
Open
@bergmeister is already working on this.
Since Feb 28, 2018.
Area - Build
Up-for-Grabs
- Dominant language
- C#
- Stars
- 2.2k
- Forks
- 414
- Avg merge
- 13h 1m
- Merged PRs (30d)
- 2
Description
UseCompatibleCmdlets produces a compiler warning when being compiled in Release mode:
UseCompatibleCmdlets.cs(38,27): warning CS0649: Field 'UseCompatibleCmdlets.RuleParameters.mode' is never assigned to, and will always have its default value null [C:\projects\psscriptanalyzer\Rules\Rules.csproj]
This seems to be due a 2 year old, unfinished feature that littered lots of #if DEBUG pragmas in this class. The blame does not tell me a referenced issue, therefore I would argue to remove this compile conditional code.
The benefit of this cleanup would also be that we could use the /warnaserror compiler option for a more rigorous test of code changes in CI.
What do you think @JamesWTruher ?
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.