Skip to content

ci(ruff): move pre-commit to ruff-check#15

Merged
JonZeolla merged 2 commits intomainfrom
update-ruff
Jul 4, 2025
Merged

ci(ruff): move pre-commit to ruff-check#15
JonZeolla merged 2 commits intomainfrom
update-ruff

Conversation

@JonZeolla
Copy link
Copy Markdown
Member

@JonZeolla JonZeolla commented Jul 4, 2025

Contributor Comments

This moves the pre-commit config to use ruff-check which is the replacement for ruff and runs a global task update

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

@JonZeolla JonZeolla enabled auto-merge (squash) July 4, 2025 03:30
Copy link
Copy Markdown

@ai-coding-guardrails ai-coding-guardrails Bot left a comment

Choose a reason for hiding this comment

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

I've got 2 comments for you to consider

The PR title and description are not accurate. Here are my suggestions:

Title: ci(ruff): upgrade to v0.12.2 and migrate to ruff-check hook

Description: # Contributor Comments

This updates the pre-commit configuration for ruff:

  • Upgrades ruff from v0.12.1 to v0.12.2
  • Migrates from the deprecated ruff hook ID to the new ruff-check hook ID
  • Updates other repository versions to newer nightly builds

The ruff-check hook is the replacement for the deprecated ruff hook.

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.
List of skipped files

Reviewed with 🤟 by Zenable

Comment thread .pre-commit-config.yaml
@JonZeolla JonZeolla merged commit 1726d91 into main Jul 4, 2025
8 checks passed
@JonZeolla JonZeolla deleted the update-ruff branch July 4, 2025 12:24
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.

1 participant