Brace-based template arguments crash literal parsing with TypeError
評価
調査の方向性
parser.DefaultParseValueから始めて、既存のリテラル解析時の例外処理と関連するテストを調べてください。'{{name}}'、'{[1]: 2}'、'{[1]}'の回帰テストを追加し、有効な辞書リテラルと集合リテラルが引き続き解析され、無効な入力では元の文字列が返されることを確認してください。
索引モデルが issue の本文から書いたものです。
説明
parser.DefaultParseValue('{{name}}') raises TypeError: unhashable type: 'set' on current master. The same happens for {[1]: 2} and {[1]}. These inputs should fall back to the original string, as other values that cannot be parsed as literals do. This currently breaks CLI arguments containing brace-based templates.
I plan to handle TypeError alongside the existing literal-parsing exceptions and add regression coverage, while preserving valid dict/set parsing.
- 主要言語
- Python
- スター
- 28.2k
- フォーク
- 1.5k
- PR マージ指標
- 30日以内にマージされた PR はありません
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
google/python-fire のほかの issue
-
WrappedJoin adds an empty line before an oversized first item対応中かも @GreedyC が 5 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
google/python-fire#705 ·
-
Test suite fails on Windows: unescaped path regex in testArgPassing, NamedTemporaryFile reopen in testFileNameFire対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
google/python-fire#693 ·
-
Mio ❤️オープン
難易度 5/5 1週間以上 初心者へのやさしさ 1/100
google/python-fire#709 · コメント 2 件 ·
-
2083オープン
難易度 5/5 1週間以上 初心者へのやさしさ 5/100
google/python-fire#708 ·
-
難易度 2/5 1〜3時間 初心者へのやさしさ 45/100
google/python-fire#707 ·
google/python-fire の issue をすべて見る
似ている issue
-
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
-
Harmony OPeNDAP SubSetter (HOSS) Geographic LARC_CLOUD PREFIRE_SAT2_AUX-SAT R01 production
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
nasa/harmony-autotester#245 ·
-
enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Deltares/imod-python#1928 ·
-
難易度 1/5 1時間未満 初心者へのやさしさ 88/100
メンテナーはふだん 1 日以内に返信
-
feature
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100