This change addresses review comments left by @ErikBjare on PR #33: feat: add toml config and arg parsing for poll time
:warning: We couldn't build/test your project to verify our changes. Add a Dockerfile to significantly improve code quality.
Summary:
Refactor main function in src/aw_watcher_input/main.py to use click for CLI handling and fix poll_time references.
Key points:
Refactor src/aw_watcher_input/main.py to use click for CLI handling.
Remove args parameter from main function.
Introduce --testing flag in main function.
Fix variable references for poll_time in main function.
You can configure Ellipsis to address comments with a direct commit or a side PR, see docs.
Something look wrong?: If this Pull Request doesn't address the comments left on the above pull request, create a new PR review with more details. For more information, check the documentation.
This change addresses review comments left by @ErikBjare on PR #33: feat: add toml config and arg parsing for poll time
Summary:
Refactor
main
function insrc/aw_watcher_input/main.py
to useclick
for CLI handling and fixpoll_time
references.Key points:
src/aw_watcher_input/main.py
to useclick
for CLI handling.args
parameter frommain
function.--testing
flag inmain
function.poll_time
inmain
function.You can configure Ellipsis to address comments with a direct commit or a side PR, see docs.
Something look wrong?: If this Pull Request doesn't address the comments left on the above pull request, create a new PR review with more details. For more information, check the documentation.
Generated with :heart: by ellipsis.dev