Skip to content

Conversation

dmarek-flex
Copy link
Contributor

@dmarek-flex dmarek-flex commented Oct 15, 2025

Note sure if this affected anyone else, but I was getting duplicate ruff checks in the pre-commit hook.

Greptile Overview

Updated On: 2025-10-15 20:49:00 UTC

Greptile Summary

This PR addresses a pre-commit configuration issue where ruff code quality checks were running multiple times. The change restricts both ruff-check and ruff-format hooks to only execute during the pre-commit stage by adding stages: [pre-commit] to both hooks in .pre-commit-config.yaml. This follows the same pattern already used by the commitlint hook and prevents the duplicate execution that was causing the issue described in the PR. The change is minimal and targeted, affecting only the pre-commit hook configuration without impacting the actual code quality checks or formatting behavior.

PR Description Notes:

  • Minor typo: "Note sure" should be "Not sure"

Important Files Changed

Changed Files
Filename Score Overview
.pre-commit-config.yaml 5/5 Added stage restrictions to ruff hooks to prevent duplicate execution

Confidence score: 5/5

  • This PR is safe to merge with minimal risk
  • Score reflects a simple, well-understood configuration change that follows established patterns and resolves a known issue without affecting functionality
  • No files require special attention

@dmarek-flex dmarek-flex self-assigned this Oct 15, 2025
Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@dmarek-flex dmarek-flex deleted the dmarek/remove-duplicate-ruff-checks branch October 15, 2025 20:53
Copy link
Contributor

Diff Coverage

Diff: origin/develop...HEAD, staged and unstaged changes

No lines with coverage information in this diff.

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