Skip to content

chore: use the non-legacy ruff check pre-commit hook - #562

Merged
lengau merged 1 commit into
mainfrom
work/non-legacy-ruff-pre-commit
Jun 15, 2026
Merged

chore: use the non-legacy ruff check pre-commit hook#562
lengau merged 1 commit into
mainfrom
work/non-legacy-ruff-pre-commit

Conversation

@lengau

@lengau lengau commented Jun 15, 2026

Copy link
Copy Markdown
Collaborator

The ruff pre-commit hook is a legacy one replaced with ruff-check.


  • I've followed the contribution guidelines.
  • I've signed the CLA.
  • I've successfully run make lint && make test.
  • I've added or updated any relevant documentation.
  • In documents I changed, I added a meta description if one was missing.
  • I've updated the relevant release notes.

Signed-off-by: Alex M. Lowe <alex.lowe@canonical.com>
@lengau
lengau requested review from a team, bepri and Copilot June 15, 2026 12:42

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 updates the project’s pre-commit configuration to use Ruff’s non-legacy lint hook (ruff-check) instead of the legacy ruff hook, aligning the repo with the current astral-sh/ruff-pre-commit hook naming.

Changes:

  • Switch the Ruff linter pre-commit hook from ruff to ruff-check while keeping the same lint autofix arguments.
  • Keep the Ruff formatter hook (ruff-format) unchanged.

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

@lengau
lengau merged commit ba42dd9 into main Jun 15, 2026
13 checks passed
@lengau
lengau deleted the work/non-legacy-ruff-pre-commit branch June 15, 2026 16:30
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.

4 participants