microsoft / microsoft/TypeScript-Sublime-Plugin
Proposal to add a global option for typescript_tsdk
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.7k
- Forks
- 236
- Ø Merge
- 21 T. 13 Std.
- Gemergte PRs (30 T.)
- 1
Beschreibung
This is somewhat related to issue 499.
Like some other users, I share user settings between machines, and the path to the global TypeScript install is different on each machine. It would be nice if I could set typescript_tsdk: global, or something similar, and the plugin would automatically look for my global install.
I'd be willing to do a pull request. It seems like you could just add another case to the initialize method in editor_client.py. It could look like this:
if tsdk_location == "global":
# Get location of npm packages by running `npm config get prefix`
# See https://stackoverflow.com/questions/4760215/running-shell-command-from-python-and-capturing-the-output
# From there, get path to global TypeScript install.
# Set variables
elif tsdk_location:
# Keep the rest the same
Thoughts?
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
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
Lies zunächst die initialize-Methode in typescript/libs/editor_client.py und den zugehörigen Issue 499. Prüfe, wie das Plugin derzeit mit typescript_tsdk umgeht, und verifiziere anschließend, wie npm config get prefix die globale TypeScript-Installation ermittelt. Als erledigt gilt die Aufgabe, wenn eine globale Einstellung die Installation auflöst und vorhandene explizite Pfade weiterhin funktionieren.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- node.js, python, typescript
- Bereich
- tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100