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

Modernize cookiecutter #121

Merged
merged 7 commits into from
Sep 17, 2024
Merged

Modernize cookiecutter #121

merged 7 commits into from
Sep 17, 2024

Conversation

Zeitsperre
Copy link
Collaborator

@Zeitsperre Zeitsperre commented Sep 16, 2024

Overview

Changes:

  • Replaced setup.py with pyproject.toml with differing backends for the top-level and templates (setuptools and flit)
  • Removed many obsolete configuration files
  • Updated GitHub Actions versions to their latest commits
  • Dropped support for Python older than version 3.9
  • Added pre-commit configuration to the template
  • Updated the licensing to add a preamble to the __init__.py
  • Replaced bump2version with bump-my-version
  • Modernizes Python code to be Python3.9+ compliant

Related Issue / Discussion

https://peps.python.org/pep-0517/
https://flit.pypa.io/en/stable/
https://pre-commit.com/

…, adapt flit for template, use ruff instead of flake8, use bump-my-version, pin dependencies, drop support for Python below 3.9, documentation fixes
@Zeitsperre Zeitsperre self-assigned this Sep 16, 2024
@Zeitsperre Zeitsperre requested a review from huard September 17, 2024 14:55
@Zeitsperre Zeitsperre marked this pull request as ready for review September 17, 2024 14:56
Copy link
Collaborator

@huard huard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Far from an expert in repo config though.

@Zeitsperre Zeitsperre merged commit 98b9777 into upstream-changes Sep 17, 2024
6 checks passed
@Zeitsperre Zeitsperre deleted the modernize branch September 17, 2024 15:44
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