-
Notifications
You must be signed in to change notification settings - Fork 77
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
Switch to use pyproject.toml instead of setup.py, and skip versioneer #169
Conversation
Signed-off-by: Adam.Dybbroe <[email protected]>
Signed-off-by: Adam.Dybbroe <[email protected]>
Signed-off-by: Adam.Dybbroe <[email protected]>
Signed-off-by: Adam.Dybbroe <[email protected]>
Signed-off-by: Adam.Dybbroe <[email protected]>
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #169 +/- ##
==========================================
- Coverage 88.12% 87.94% -0.18%
==========================================
Files 14 16 +2
Lines 2265 2306 +41
==========================================
+ Hits 1996 2028 +32
- Misses 269 278 +9
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
Signed-off-by: Adam.Dybbroe <[email protected]>
…tal into switch-to-pyproject-toml
Signed-off-by: Adam.Dybbroe <[email protected]>
…tal into switch-to-pyproject-toml
Signed-off-by: Adam.Dybbroe <[email protected]>
…tal into switch-to-pyproject-toml
Signed-off-by: Adam.Dybbroe <[email protected]>
Signed-off-by: Adam.Dybbroe <[email protected]>
As discussed the one remaining ruff error was a complexity issue with |
I'm actually already browsing through |
This PR started out solving a deprecation warning when installing with pip:
It ended up switching to pyproject.toml, ditching versioneer.py and by using more strict style checkers with ruff many small issues needed attention.
flake8 pyorbital