Skip to content
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

Modernization #267

Merged
merged 14 commits into from
Mar 6, 2025
Merged

Modernization #267

merged 14 commits into from
Mar 6, 2025

Conversation

Zeitsperre
Copy link
Contributor

Overview

Changes:

  • Added support for Python3.12 and Python3.13
  • Updated the authors file and Zenodo configuration to reflect contributors.
  • Replaced bumpversion (obsolete) with bump-my-version.
  • Added dependencies that were missing from the requirements*.txt and environment*.yml files.
  • Set base versions for most dependencies.
  • Updated the build setup for ReadTheDocs.
  • Updated the pre-commit hook versions.
  • Added security hardening steps to workflows.
  • Replaced pydocstyle with numpydoc-validation
  • Added missing docstrings for all public functions, classes, and methods.

Related Issue / Discussion

Static typing was inferred by me, so there may be errors and inaccuracies.

Additional Information

I may have gone a bit overboard here, but there is a need for us to support modern Python versions.

@Zeitsperre Zeitsperre requested a review from cehbrecht March 4, 2025 17:09
@Zeitsperre Zeitsperre self-assigned this Mar 4, 2025
@Zeitsperre
Copy link
Contributor Author

@cehbrecht This is good to go. The next step would be to translate the setup to pyproject.toml (pip >=25.0 is warning that packages should start moving to the new standard).

@Zeitsperre Zeitsperre requested a review from huard March 5, 2025 17:06
@cehbrecht
Copy link
Member

@Zeitsperre looks all good to me :)

@Zeitsperre Zeitsperre merged commit 1ffc4c1 into master Mar 6, 2025
12 checks passed
@Zeitsperre Zeitsperre deleted the update-dependencies branch March 6, 2025 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants