microsoft / microsoft/vscode-cpptools

Switch back to using the official `editorconfig` module

オープン
#13,680 コメント 6 件 リアクション 0 件 担当者 0 名 GitHub で見る

まだ誰も着手していません。

Language Service
主要言語
TypeScript
スター
6.2k
フォーク
1.7k
平均マージ
14時間 46分
マージ済み PR(30日)
61

説明

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.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

調査の方向性

拡張機能内で複製された .editorconfig パーサーと現在の依存関係宣言を検索し、リンクされている editorconfig-core-js issue と過去の pull request を確認して、元の失敗とその修正を調査します。完了条件は、報告されたシステム固有の依存関係問題を再現することなく、公式の editorconfig モジュールが再び使用され、既存の解析動作が維持されていることです。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
typescript
領域
tooling
issue の種類
リファクタリング
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。