-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Switch to ruff linter and formatter #422
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #422 +/- ##
=======================================
Coverage 60.09% 60.09%
=======================================
Files 23 23
Lines 1466 1466
=======================================
Hits 881 881
Misses 585 585
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -5,7 +5,7 @@ updates: | |||
- package-ecosystem: github-actions | |||
directory: / | |||
schedule: | |||
interval: daily | |||
interval: monthly |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This seems a bit saner and matches what we have in AWB.
except RuntimeError as inner_error: | ||
logger.error(f"Rollback failed due to error: {inner_error}!") | ||
except RuntimeError: | ||
logger.exception("Rollback failed! Consider re-installing the app.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Apparently, the exception is always automatically included when using logger.exception
inside except statement.
@@ -121,7 +122,6 @@ def build( | |||
api_path=base_path / api_path, | |||
apps_index=apps_index, | |||
apps_data=apps_data, | |||
scan_app_repository=parse_app_repo, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This argument was completely unused in build_api_v1
, perhaps copy-paste error or some historical remnant?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
I've taken ruff config from AWB (with minor modifications as needed).