PowerShell / PowerShell/PSScriptAnalyzer
Windows should not be a plural noun
Nessuno ha ancora preso questa issue.
- Lingua principale
- C#
- Stelle
- 2.2k
- Fork
- 414
- Merge medio
- 13h 1m
- PR unite (30g)
- 2
Descrizione
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
Guida per i contributori
Apri la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia riproducendo l'avviso PSUseSingularNouns con la funzione Test-Windows nelle versioni di PowerShell segnalate, quindi traccia la regola PSUseSingularNouns nell'analizzatore. Il lavoro è completato quando l'avviso non viene emesso per il sostantivo Windows, mentre la regola continua a segnalare i sostantivi effettivamente plurali.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- csharp, powershell
- Ambito
- tooling
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 45/100