-
Notifications
You must be signed in to change notification settings - Fork 99
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
track improvements towards pep8 compliance #493
base: development
Are you sure you want to change the base?
Conversation
👈 Launch a binder notebook on this branch for commit ba8c5b6 I will automatically update this comment whenever this PR is modified 👈 Launch a binder notebook on this branch for commit a2ad04b 👈 Launch a binder notebook on this branch for commit 26d8f4c 👈 Launch a binder notebook on this branch for commit aa79e39 👈 Launch a binder notebook on this branch for commit 99b58a5 👈 Launch a binder notebook on this branch for commit 986555e 👈 Launch a binder notebook on this branch for commit c021602 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development #493 +/- ##
============================================
Coverage 66.59% 66.59%
============================================
Files 36 36
Lines 3059 3059
Branches 534 534
============================================
Hits 2037 2037
Misses 934 934
Partials 88 88 ☔ View full report in Codecov by Sentry. |
for more information, see https://pre-commit.ci
After updating the GH actions (#475), in order to get the checks to pass we needed to add a lot of ignore codes to the .flake8 config file. This draft PR removes those ignores so PRs moving forward have a spot to look to see if any edited files are out of compliance (but the checks will still pass as we work towards compliance for legacy code).