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

Update pre commit, lint files and fix docs #39

Open
wants to merge 26 commits into
base: develop
Choose a base branch
from

Conversation

mikemanger
Copy link

@mikemanger mikemanger commented Jul 29, 2024

  • Switches out various pre-commit tasks to ruff to (fixes Pre-commit tasks not working #37)
  • Adds linter job to workflow
  • Documentation can be compiled again (fixes Docs not compiling #38)
  • Added __str__() definitions for models
  • Add Python 3.7 and 3.12 to the classifiers
  • Remove unsupported Django and Python code
  • Update setup guide and use https links
  • Use path instead of re_path for most views

I've ported most of the old flake8/black/isort config to ruff and added flake8-django to the linting config. I've run this config on all files which only brought out one minor user facing change: adding __str__() definitions to the models.

The changes are minor so I'm not sure it warrants a new release however it might be good to update the documentation pages as they're currently out of date.

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.

Docs not compiling Pre-commit tasks not working
1 participant