adobe / adobe/aio-cli-plugin-runtime

enhance action create with env variables

オープン
#116 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement help wanted
主要言語
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

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

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