Skip to content

Introduce pre-commit configuration#280

Merged
bgunebakan merged 2 commits into
mainfrom
279-introduce-pre-commit-to-check-linting-before-commit
Jun 19, 2026
Merged

Introduce pre-commit configuration#280
bgunebakan merged 2 commits into
mainfrom
279-introduce-pre-commit-to-check-linting-before-commit

Conversation

@bgunebakan

@bgunebakan bgunebakan commented Jun 19, 2026

Copy link
Copy Markdown
Contributor

Summary of the changes / Why this is an improvement

Checklist

@bgunebakan bgunebakan self-assigned this Jun 19, 2026
@bgunebakan bgunebakan added the enhancement New feature or request label Jun 19, 2026
@bgunebakan bgunebakan linked an issue Jun 19, 2026 that may be closed by this pull request
@coderabbitai

coderabbitai Bot commented Jun 19, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 34920daa-ea2f-478f-9283-189ee5e0384a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch 279-introduce-pre-commit-to-check-linting-before-commit

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bgunebakan bgunebakan requested review from florinutz and kneth June 19, 2026 11:37
Comment thread .pre-commit-config.yaml
Comment on lines +3 to +4
- repo: https://github.com/astral-sh/ruff-pre-commit
rev: v0.15.13

@amotl amotl Jun 19, 2026

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.

Hi. It is difficult to keep the version in sync with the version written in pyproject.toml, right? Maybe there is a better solution for that today?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Unfortunately no, pre-commit config does not support dynamic version definition.

@bgunebakan bgunebakan merged commit d4b5ae9 into main Jun 19, 2026
27 checks passed
@bgunebakan bgunebakan deleted the 279-introduce-pre-commit-to-check-linting-before-commit branch June 19, 2026 14:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Introduce pre-commit to check linting before commit

3 participants