PowerShell / PowerShell/PSScriptAnalyzer
Windows should not be a plural noun
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- C#
- Sterne
- 2.2k
- Forks
- 414
- Ø Merge
- 13 Std. 1 Min.
- Gemergte PRs (30 T.)
- 2
Beschreibung
Before submitting a bug report:
- Make sure you are able to repro it on the latest released version
- Perform a quick search for existing issues to check if this bug has already been reported
Steps to reproduce
See this tweet https://twitter.com/FrankLesniak/status/1453189852032344064?s=20
function Test-Windows {
dostuff
}
Expected behavior
PSUseSingularNouns warning is not thrown when noun finishes with Windows
Actual behavior
PSUseSingularNouns warning thrown
If an unexpected error was thrown then please report the full error details using e.g. $error[0] | Select-Object *
Environment data
Name Value
---- -----
PSVersion 7.2.0-preview.10
PSEdition Core
GitCommitId 7.2.0-preview.10
OS Microsoft Windows 10.0.19043
Platform Win32NT
PSCompatibleVersions {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion 2.3
SerializationVersion 1.1.0.1
WSManStackVersion 3.0
1.18.3 - Doesn't throw
1.20.0 - Does throw
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die PSUseSingularNouns-Warnung mit der Test-Windows-Funktion auf den gemeldeten PowerShell-Versionen zu reproduzieren, und verfolge dann die PSUseSingularNouns-Regel im Analyzer. Als abgeschlossen gilt die Aufgabe, wenn die Warnung für das Substantiv Windows nicht ausgegeben wird, während die Regel weiterhin tatsächlich pluralische Substantive meldet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- csharp, powershell
- Bereich
- tooling
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100