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

Ouverte
#913 0 commentaires 0 réactions 1 personne assignée Voir sur GitHub

@bergmeister y travaille déjà.

Depuis le 28/2/2018.

Area - Build Up-for-Grabs
Langage dominant
C#
Étoiles
2.2k
Forks
414
Merge moyen
13 h 1 min
PR mergées (30 j)
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 ?

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.