PowerShell / PowerShell/PSScriptAnalyzer

Flag usage of >< etc when -gt, -lt, etc should be used

Offen
#1,900 2 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Issue-Enhancement Up-for-Grabs
Vorherrschende Sprache
C#
Sterne
2.2k
Forks
414
Ø Merge
13 Std. 1 Min.
Gemergte PRs (30 T.)
2

Beschreibung

Prerequisites
  • I have written a descriptive issue title.
  • I have searched all issues to ensure it has not already been reported.
Summary

I would like the Visual Studio Code editor to flag usage of >< !=, etc as possibly incorrect if used in a conditional.
I go back and forth between languages and I inadvertently use C#/TypeScript syntax in conditionals and then it takes a while to find my error. An error/warning/hint that I am probably using the incorrect syntax would be great.

Proposed Design

No response

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne mit der Prüfung von Issue #1900 und der PSScriptAnalyzer-Analyzer-Einstiegspunkte, die PowerShell-Bedingungsausdrücke untersuchen; der Payload nennt keine bestimmte Datei oder keinen bestimmten Test. Ermittle, wie eine Diagnose diese Operatoren in Bedingungen unterscheiden sollte, füge dann eine Abdeckung hinzu, die die beabsichtigte Warnung oder den Hinweis zeigt, und verifiziere, dass der Analyzer sie meldet, ohne gültige Syntax zu markieren.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
csharp, powershell, vscode
Bereich
devtools, tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.