-
Notifications
You must be signed in to change notification settings - Fork 666
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 analysis module and tests #4848
Conversation
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.
blanket LGTM
🎄 🎁
I turned off formatting for some ugly |
@marinegor this should be good for a final review, I think. |
Merging to avoid other PRs getting in the way. @marinegor please review regardless. If you spot something off, we can fix it retrospectively. |
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.
retrospective LGTM
Format most of the
MDAnalysis/analysis
module and corresponding test. Some files actives in current PRs are ignored and will be formatted once the corresponding PRs are merged (see #4815).📚 Documentation preview 📚: https://mdanalysis--4848.org.readthedocs.build/en/4848/