icesat2py / icepyx

Python tools for obtaining and working with ICESat-2 data
https://icepyx.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
201 stars 100 forks source link

track improvements towards pep8 compliance #493

Open JessicaS11 opened 6 months ago

JessicaS11 commented 6 months ago

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).

github-actions[bot] commented 6 months ago

Binder :point_left: Launch a binder notebook on this branch for commit ba8c5b60c915f6095148979037633355fabb7d43

I will automatically update this comment whenever this PR is modified

Binder :point_left: Launch a binder notebook on this branch for commit a2ad04b8a48381d9d7f05373492b8608100266a0

Binder :point_left: Launch a binder notebook on this branch for commit 26d8f4c2ec9b29c5234b4e15ba23fd2b1691e933

Binder :point_left: Launch a binder notebook on this branch for commit aa79e396238760dc0eba38a754f045b587140eac

Binder :point_left: Launch a binder notebook on this branch for commit 99b58a58a412b920334a82946e80cebe5562f01e

Binder :point_left: Launch a binder notebook on this branch for commit 986555ed24b6b8fc7d7bcb71240e3263fb21101a

Binder :point_left: Launch a binder notebook on this branch for commit c021602787a45d72d90c5db9e7db0c6f2c5b624d

Binder :point_left: Launch a binder notebook on this branch for commit 104ee26cb38a4ddec4c5280912e50787377a61e2

Binder :point_left: Launch a binder notebook on this branch for commit b19907f071a2ffe9da46521f809468db3cb2c38c

Binder :point_left: Launch a binder notebook on this branch for commit eb95a4237b49c1855f9cda99ced9e39d1b3e0ef7

codecov[bot] commented 5 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 66.27%. Comparing base (102405b) to head (104ee26). Report is 1 commits behind head on development.

:exclamation: Current head 104ee26 differs from pull request most recent head eb95a42

Please upload reports for the commit eb95a42 to get more accurate results.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## development #493 +/- ## ============================================ Coverage 66.27% 66.27% ============================================ Files 36 36 Lines 3072 3072 Branches 537 537 ============================================ Hits 2036 2036 Misses 948 948 Partials 88 88 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.