リンク
GitHub
Efficient deploy log fetching by smatsumt · Pull Request #1768 · awslabs/aws-sam-cli
Why is this change necessary? This PR improves sam deploy inefficient log fetching. Current implementation fetches events repeatidly until deploy finished. And it fetches "all" log events...
Shinji Matsumoto
@smatsumt
PR について「ライセンスまわりは見ておいたほうがいいよ」というアドバイスを @terapyon さんからいただき、改めて SAM CLI Contribution Guide github.com/awslabs/aws-sa… を確認してみました。(あまりちゃんと見てませんでした (^_^; ) #pyhack
2020-02-02 10:20:24
Shinji Matsumoto
@smatsumt
@terapyon ページの一番下に、 "We may ask you to sign a Contributor License Agreement (CLA) for larger changes." とあり、大きめの変更だと書類へのサインが求められるようです。 今回は小ネタなので大丈夫そうですが、プロジェクトによってサインが必要なプロジェクトとかもありそうですね。 #pyhack
2020-02-02 10:21:23
Shinji Matsumoto
@smatsumt
@terapyon 今回のプルリク作るにあたり、二重ループの中から break する必要があったので note.nkmk.me/python-break-n… を参考にさせてもらいました。こちらのブログはいつも参考にさせてもらっています。 #pyhack
2020-02-02 10:29:02
Shinji Matsumoto
@smatsumt
@terapyon 最初は "フラグ変数を追加" でやったものの、lint で「1メソッドの文が多すぎ」と怒られたので、1番目の "for の else 節を使う" に乗り換え。for の else 節はこうやって使うのか😲 でも却って読みにくいのでは疑惑、、、 #pyhack github.com/awslabs/aws-sa…
2020-02-02 10:32:32