-
Notifications
You must be signed in to change notification settings - Fork 207
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
Fixes #3023
Fixes #3023
Conversation
ouziel-slama
commented
Feb 17, 2025
•
edited
Loading
edited
- Double-check the spelling and grammar of all strings, code comments, etc.
- Double-check that all code is deterministic that needs to be
- Add tests to cover any new or revised logic
- Ensure that the test suite passes
- Update the project release notes
- Update the project documentation, as appropriate, with a corresponding Pull Request in the Documentation repository
try: | ||
check_signature(zst_filepath, sig_url) | ||
decompress_zst(zst_filepath) | ||
except Exception as e: |
Check warning
Code scanning / pylint
Catching too general exception Exception. Warning
# download .zst file | ||
try: | ||
zst_filepath = download_zst(data_dir, zst_url) | ||
except Exception as e: |
Check warning
Code scanning / pylint
Catching too general exception Exception. Warning
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3023 +/- ##
===========================================
- Coverage 80.77% 80.62% -0.16%
===========================================
Files 99 99
Lines 13331 13351 +20
===========================================
- Hits 10768 10764 -4
- Misses 2563 2587 +24 ☔ View full report in Codecov by Sentry. |