Code review comment for ~jslarraz/review-tools:run-black

Revision history for this message
Alex Murray (alexmurray) wrote :

LGTM - as seen in the lpci test log:

:: ./run-black
:: = black =
:: Checking ./reviewtools/__init__.py
:: Checking ./reviewtools/available.py
:: Checking ./reviewtools/common.py
:: Checking ./reviewtools/debversion.py
:: Checking ./reviewtools/email.py
:: Checking ./reviewtools/modules.py
:: Checking ./reviewtools/overrides.py
:: Checking ./reviewtools/rr_common.py
:: Checking ./reviewtools/sr_common.py
:: Checking ./reviewtools/sr_declaration.py
:: Checking ./reviewtools/sr_functional.py
:: Checking ./reviewtools/sr_lint.py
:: Checking ./reviewtools/sr_security.py
:: Checking ./reviewtools/sr_skeleton.py
:: Checking ./reviewtools/sr_tests.py
:: Checking ./reviewtools/store.py
:: Checking ./reviewtools/usn.py
:: Checking ./reviewtools/tests/__init__.py
:: Checking ./reviewtools/tests/test_available.py
:: Checking ./reviewtools/tests/test_bbb_example_sr_skeleton.py
:: Checking ./reviewtools/tests/test_common.py
:: Checking ./reviewtools/tests/test_debversion.py
:: Checking ./reviewtools/tests/test_email.py
:: Checking ./reviewtools/tests/test_modules.py
:: Checking ./reviewtools/tests/test_sr_common.py
:: Checking ./reviewtools/tests/test_sr_declaration.py
:: Checking ./reviewtools/tests/test_sr_functional.py
:: Checking ./reviewtools/tests/test_sr_lint.py
:: Checking ./reviewtools/tests/test_sr_security.py
:: Checking ./reviewtools/tests/test_store.py
:: Checking ./reviewtools/tests/test_usn.py
:: Checking ./reviewtools/tests/utils.py
:: Checking ./bin/snap-review
:: Checking ./bin/create-snap-declaration
:: Checking ./bin/dump-tool
:: Checking ./bin/get-base-declaration
:: Checking ./bin/store-query
:: Checking ./bin/rock-updates-available
:: Checking ./bin/snap-updates-available
:: Checking ./bin/snap-check-declaration
:: Checking ./bin/snap-check-lint
:: Checking ./bin/snap-check-security
:: Checking ./bin/snap-check-skeleton
:: Checking ./bin/snap-verify-declaration

review: Approve

« Back to merge proposal