PowerShell / PowerShell/PSScriptAnalyzer
"Reserved prefixes" rule
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
Summary of the new feature
A while ago, the PowerShell extension had an issue open on it because
Get-Help Get-AdUser -Online
and
Get-Help Get-AdFoo -Online
(note, Foo is used to make it obvious and I forget what the actual noun was)
Were not giving the same result...
As it turned out, its because Get-AdFoo was from a module from the Gallery... Where Get-AdUser comes from the inbox Active Directory module.
Personally, I think it's quite dodgy to overload such a crucial prefix like Ad and we should warn users that certain prefixes are used already in very critical modules.
Proposed technical implementation details (optional)
A rule that gives a warning that says "you probably shouldn't use this prefix because it's already used by X"
Or something to that effect.
What is the latest version of PSScriptAnalyzer at the point of writing
1.18.0
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
Es sind keine Implementierungsdateien oder Tests angegeben. Beginne damit, die vorhandenen Einstiegspunkte der PSScriptAnalyzer-Regeln zu finden und zu untersuchen, wie PowerShell-Befehlsnamen und Module dargestellt werden; kläre anschließend die Quelle des reservierten Präfixes, das Warnverhalten und die Akzeptanzkriterien, bevor du die Regel implementierst oder testest.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- csharp, powershell
- Bereich
- tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 5/5
- Geschätzter Aufwand
- Über eine Woche
- Aktivitätsstatus
- Veraltet
- Klarheit
- Muss geklärt werden
- Anfängerfreundlichkeit
- 25/100