microsoft / microsoft/vscode-cpptools
Switch back to using the official `editorconfig` module
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 6.2k
- Forks
- 1.7k
- Ø Merge
- 14 Std. 46 Min.
- Gemergte PRs (30 T.)
- 61
Beschreibung
We had previously been using the editorconfig module to parse .editorConfig files. Due to an issue that occurred on some systems, because of a dependency of that module, we removed it in favor of replicating the code we needed:
https://github.com/microsoft/vscode-cpptools/issues/12529
https://github.com/microsoft/vscode-cpptools/pull/12579
I also opened a bug against editorconfig core: https://github.com/editorconfig/editorconfig-core-js/issues/128
It looks like that bug is now fixed, so we may be able to switch back to using the editorconfig module.
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
Durchsuche die Erweiterung nach dem duplizierten .editorconfig-Parser und den aktuellen Abhängigkeitsdeklarationen und prüfe anschließend das verknüpfte editorconfig-core-js-Issue sowie den vorherigen Pull Request auf den ursprünglichen Fehler und dessen Behebung. Als erledigt gilt die Aufgabe, wenn wieder das offizielle editorconfig-Modul verwendet wird, ohne das gemeldete systemspezifische Abhängigkeitsproblem erneut zu reproduzieren, und das bestehende Parsing-Verhalten erhalten bleibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- typescript
- Bereich
- tooling
- Issue-Typ
- Refactoring
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 38/100