Skip to content

Conversation

@bittner
Copy link
Member

@bittner bittner commented Apr 4, 2025

Migrates the packaging setup to pyproject.toml, finally, and updates the CI setup (GHA) targeting the currently latest Django versions and their related Python versions.

  • Replaces Flake8 and isort by the Ruff linter and formatter, retaining the single-quote formatting for strings.
  • Also fixes some documentation issues and outdated URL references.
  • Removes the obsolete type="text/javascript" attribute from all script HTML tags for JavaScript. (This change is not strictly necessary for this PR and can be omitted if requested; it's included for convenience.)

Note that all authors are now listed in pyproject.toml; we now might remove the AUTHORS.rst file.

I suggest to keep the commits intact and refrain from squashing.

Obsoletes #227, #229 and #230.

@bittner bittner requested review from jezdez and smithdc1 April 4, 2025 12:26
@bittner bittner self-assigned this Apr 4, 2025
@bittner
Copy link
Member Author

bittner commented Apr 20, 2025

Happy seasonal greetings! 🐰 🪺 🥚

Review this code, anyone? – If you review the commits separately it should be a quick thing to do. Thanks!

@bittner
Copy link
Member Author

bittner commented Apr 30, 2025

Hello @jazzband! - Someone, please review! 📣

@bittner
Copy link
Member Author

bittner commented Jun 20, 2025

Hello @jezdez, hi @smithdc1, can one of you please approve this PR, so that it can be merged?

⚠️ ⚠️ ⚠️ 💣 💣 💣 😨 😨 😨

Other PRs are blocked by this, for example #235.

Copy link
Member

@matthiask matthiask left a comment

Choose a reason for hiding this comment

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

It's a lot and I havent read everything. The parts which I've read closely look good to me.

(Yep, requiring reviews is bad, but not requiring them isn't good either.)

@bittner bittner merged commit 4540999 into main Jul 8, 2025
19 checks passed
@bittner bittner deleted the feature/update-gha-checks-and-testmatrix branch July 8, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants