adobe / adobe/aio-cli-plugin-runtime
enhance action create with env variables
- 主要言語
- JavaScript
- スター
- 21
- フォーク
- 33
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
**Describe the bug**
This was missed in the recently merged PR #103. The `-P` flag is overloaded to mean read params from file and read env from file. This is a copy/paste bug, the intent was to use `-E`.
https://github.com/adobe/aio-cli-plugin-runtime/pull/103/files#diff-6d51d7a8679a906a9bfdaf1a8b017006R207
Thanks to @purplecabbage for pointing this out. We were both surprised oclif didn't barf at this.
As an enhancement, Jesse also suggested supporting reading the value of an env var from the shell environemnt, so that you can do `... action create myAction -e FOO` which is syntactic sugar for `... action create myAction -e FOO $FOO`. I think this is neat/convenient (perhaps to be considered separately as an enhancement).
コントリビューションガイド
調査の方向性
PR #103 でリンクされている箇所の action create のオプション処理を確認し、オーバーロードされた -P フラグと意図された -E フラグに焦点を当てます。既存のパラメーターファイルの動作を確認したうえで、shell 環境の検索が含まれるのか、別途扱われるのかを判断します。完了条件は、各フラグの動作が明確に異なり、要求されたコマンドの使用方法がテストでカバーされていることです。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- javascript
- 領域
- cli
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 38/100