openmmforcefields
openmmforcefields copied to clipboard
Add uniform, automated linting
I find it difficult to read some changesets in this project, owing largely to things like long lines and inconsistent text wrapping. Enter (standard set of tools). The https://pre-commit.ci/ would make this a little smoother but is not necessary.
The changes here are mostly automated, though I had to make some manual fixes and I've also nested a few questions inside the code itself.
Happy to take responsibility for rebasing other PRs.
I will merge this in once I get the other PRs merged in, run the pre-commit on the whole code base, squash in the commits, then add a file so the web-ui knows to skip that commit in the blame
just updated the oe license and re-started CI
Merging upstream changing in now ...
Codecov Report
Attention: Patch coverage is 42.67782% with 137 lines in your changes are missing coverage. Please review.
Project coverage is 52.98%. Comparing base (
016c836) to head (85a7b7f).
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@ Coverage Diff @@
## main #319 +/- ##
==========================================
+ Coverage 52.80% 52.98% +0.18%
==========================================
Files 5 5
Lines 820 804 -16
==========================================
- Hits 433 426 -7
+ Misses 387 378 -9
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.