tomomano / learn-aws-by-coding

コードで学ぶAWS入門
395 stars 38 forks source link

8章の run_task.py の --limit の引数が使われていない #37

Closed msakai closed 2 years ago

msakai commented 2 years ago

「8. Hands-on #3: AWS で自動質問回答ボットを走らせる」の handson/qa-bot/run_task.pylist_answers コマンドには --limit オプションが用意されていますが、list_answers 関数の実装では limit 引数は使用されていないようです。

https://github.com/tomomano/learn-aws-by-coding/blob/05a1d8fd32b0d6b6d503b271874f2c2436a8f068/handson/qa-bot/run_task.py#L163-L164

https://github.com/tomomano/learn-aws-by-coding/blob/05a1d8fd32b0d6b6d503b271874f2c2436a8f068/handson/qa-bot/run_task.py#L174-L175

https://github.com/tomomano/learn-aws-by-coding/blob/05a1d8fd32b0d6b6d503b271874f2c2436a8f068/handson/qa-bot/run_task.py#L116-L131

tomomano commented 2 years ago

@msakai

細かいところまで気がついていただき,どうもありがとうございます. 確かに, limit の引数が使用されていませんでした.

39 のPRにて修正しました.

https://github.com/tomomano/learn-aws-by-coding/pull/39/files

(コード見直して気が付きましたが,limit パラメータつけてみたものの, ExclusiveStartKey を使って再帰的に呼び出しているわけでもないし,オフセットも指定できるわけでもないので,実質あんまり使い物にならないですね...笑 なんにせよありがとうございます)