speee / webapp-revieee

MIT License
39 stars 2 forks source link

Settings .envrc for using aws sdk in local environment #33

Closed selmertsx closed 7 years ago

selmertsx commented 7 years ago

手元でAWS sdkの利用をするために、envrcを入れてみました!

selmertsx commented 7 years ago

@pataiji @kou 手元でaws sdkを使うために、envrcを入れてみました。 レビューお願いします!

pataiji commented 7 years ago

依存するツールをあまり増やしたくない & 開発環境でだけ(必須ではなく)使うもの、なので、できれば.envrcは含めたくないんですがどうでしょうか?

ツールを動かす上で必要な環境変数がある場合はReadme等に書いておいて、direnvは個人の好みで使用する、という形がいいです:pray:

selmertsx commented 7 years ago

依存するツールをあまり増やしたくない & 開発環境でだけ(必須ではなく)使うもの

ここは僕も同意っす!なので、今まで悩みつつも入れてませんでした....。 が、ちょっと手間だなぁとなって一旦入れたみた感じっす。

ということで、.gitignore にenvrc関連の設定を入れて、 みんな好きにしてって形にしようかなと思うのですがどうでしょ?

pataiji commented 7 years ago

良いと思いまする!

selmertsx commented 7 years ago

ういっす、じゃぁそれで!

selmertsx commented 7 years ago

@pataiji ということで、gitignoreに追加しました!レビューおなしゃすー

kou commented 7 years ago

よさそうな落とし所を見つけていていいなぁと思いました!

pataiji commented 7 years ago

LGTM!:+1:

selmertsx commented 7 years ago

レビューありがとうございましたー 😄