Skip to content

Conversation

@codingjoe
Copy link
Member

No description provided.

@codingjoe codingjoe self-assigned this Nov 3, 2025
@codingjoe codingjoe enabled auto-merge (rebase) November 3, 2025 11:21
Copilot AI review requested due to automatic review settings November 3, 2025 11:30
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR modernizes the project's linting and formatting infrastructure by migrating from a local lint setup to pre-commit CI, consolidating configuration, and applying formatting standards across documentation and code files.

  • Removes standalone lint job from CI workflow and optional lint dependencies from pyproject.toml
  • Migrates from local pre-commit hooks to standardized remote hooks with added formatters (mdformat, yamlfmt, pyupgrade, django-upgrade)
  • Applies formatting changes to documentation (CODE_OF_CONDUCT.md), configuration files (README.md, settings.py, YAML workflows), and removes deprecated setup.cfg

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Removes optional lint dependencies (black, pre-commit, ruff)
setup.cfg Deletes deprecated flake8 configuration file
.pre-commit-config.yaml Replaces local hooks with remote pre-commit hooks, adds new formatters, and configures pre-commit CI
.github/workflows/ci.yml Removes standalone lint job and reformats YAML structure
.github/workflows/release.yml Reformats YAML structure for consistency
tests/testapp/settings.py Simplifies SECRET_KEY assignment to single line
README.md Changes string quotes from single to double
CODE_OF_CONDUCT.md Converts bullet points from * to - and repositions homepage reference

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Member

@amureki amureki left a comment

Choose a reason for hiding this comment

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

Love it!

@amureki amureki disabled auto-merge November 3, 2025 12:31
@amureki amureki merged commit 6139e84 into main Nov 3, 2025
19 checks passed
@amureki amureki deleted the pre-commit branch November 3, 2025 12:31
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.

3 participants