Non-enum CLI options with multiple values separated by comma and space aren't whitespace trimmed
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 68/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 活発
- 技術スタック
- typescript
調査の方向性
提供された npx tsc --showConfig コマンドで検証される CLI オプション解析から始め、non-enum オプションと lib の処理を比較します。報告されたオプションがカンマ分割後に先頭のスペースを保持しなくなっていることを確認し、その一方で解決済みの構成がそれ以外は変更されていないことを確認します。
索引モデルが issue の本文から書いたものです。
説明
🔎 Search Terms
CLI, compilerOptions, whitespace, comma
🕗 Version & Regression Information
Checked on the latest main
⏯ Playground Link
No response
💻 Code
> npx tsc --showConfig --customConditions "a, b" --types "a, b" --typeRoots "a, b" --moduleSuffixes "a, b" --excludeDirectories "a, b" --excludeFiles "a, b" --lib "es2015, dom" index.ts
🙁 Actual behavior
{
"compilerOptions": {
"customConditions": [
"a",
" b"
],
"lib": [
"es6",
"dom"
],
"moduleSuffixes": [
"a",
" b"
],
"typeRoots": [
"./a",
"./ b"
],
"types": [
"a",
" b"
]
},
"files": [
"./index.ts"
]
}
See space before b in resolved config, and also note that lib values are trimmed correctly
🙂 Expected behavior
{
"compilerOptions": {
"customConditions": [
"a",
"b"
],
"lib": [
"es6",
"dom"
],
"moduleSuffixes": [
"a",
"b"
],
"typeRoots": [
"./a",
"./b"
],
"types": [
"a",
"b"
]
},
"files": [
"./index.ts"
]
}
Non-enum options should be whitespace-trimmed as well
Additional information about the issue
Followup for https://github.com/microsoft/TypeScript/pull/64522#discussion_r4147589423
- 主要言語
- Go
- スター
- 111k
- フォーク
- 15.5k
- 平均マージ
- 1日 18時間
- マージ済み PR(30日)
- 156
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
microsoft/TypeScript のほかの issue
-
Class names `eval` and `arguments` are not reported as invalid strict mode bindings対応中かも @dmety が 1 日前に担当しました。 オープンBug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
microsoft/TypeScript#64661 ·
メンテナーはふだん 1 日以内に返信
-
TypeScript 7 VS Code extension: a workspace "typescript" 7.x package is not detected, only "@typescript/native-preview", which is no longer published対応中かも @jakebailey が 2 日前に担当しました。 オープンNeeds Investigation
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
microsoft/TypeScript#64565 · リアクション 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
`EFNoLeadingComments` suppresses synthesized leading comments in tsgo; Strada only suppresses source comments対応中かも @weswigham が 9 日前に担当しました。 オープンNeeds Investigation
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
microsoft/TypeScript#64453 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
enhance: Add tests for premature caching of contextual parameter types対応中かも @luchenxu73 が 23 日前に担当しました。 オープンPossible Improvement
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
microsoft/TypeScript#64278 · コメント 1 件 · リアクション 1 件 ·
メンテナーはふだん 1 日以内に返信
-
dom.generated.d.ts is missing from this repo while still referenced in the docs再び着手できるかも @seanxuu が 36 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンDocs
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
microsoft/TypeScript#64118 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
microsoft/TypeScript の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
agent-research agent-review-finding chore
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
jordansmall/spindrift#4922 ·
メンテナーはふだん 1 日以内に返信
-
gcsartifact: deleting a missing version returns an error対応中かも @ktsoator が今日担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
メンテナーはふだん 2 日以内に返信
-
govulncheck
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
Change wording for init command success message対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 1/5 1時間未満 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信