PowerShell / PowerShell/vscode-powershell
Put the setLanguageConfiguration code in the appropriate place
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 1.9k
- Forks
- 548
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
Currently main.ts is mostly populated with this mystifying code blob:
https://github.com/PowerShell/vscode-powershell/blob/6c96a2d56b9593a25e6960df2c6f3767b3d30177/src/main.ts#L53-L104
It seems to have some PowerShell related bits in it, but also some JS doc comment stuff...
It probably shouldn't be in main.ts, and we should work out if it even causes anything. I suspect it's getting ignored in favour of the code in https://github.com/Microsoft/vscode/blob/master/extensions/powershell/language-configuration.json.
We should investigate and:
- if the code here is being preferred, move it to a more sensible location in the repo and make sure it's up to do
- if the VSCode JSON is being preferred, delete this code
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
Beginnen Sie mit dem setLanguageConfiguration-Code in src/main.ts an den verlinkten Zeilen und vergleichen Sie ihn anschließend mit der im Issue referenzierten VS Code language-configuration.json. Ermitteln Sie, welche Konfiguration verwendet wird und ob der TypeScript-Code einen Effekt hat. Die Aufgabe ist erledigt, wenn der Code entweder an eine geeignetere Stelle verschoben und aktualisiert wird oder entfernt wird, falls die JSON-Datei bevorzugt wird.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- powershell, typescript, vscode
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100