-
Notifications
You must be signed in to change notification settings - Fork 660
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
[fmt] Format converters modules and tests #4850
Conversation
Hello @RMeli! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-12-20 22:28:31 UTC |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #4850 +/- ##
========================================
Coverage 93.65% 93.65%
========================================
Files 177 177
Lines 21779 21779
Branches 3064 3064
========================================
Hits 20398 20398
Misses 929 929
Partials 452 452 ☔ View full report in Codecov by Sentry. |
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.
[fmt] Format converters modules and tests. Some files actives in current PRs are ignored and will be formatted once the corresponding PRs are merged (see #4815).
📚 Documentation preview 📚: https://mdanalysis--4850.org.readthedocs.build/en/4850/