Skip to content

Fix/pre-commit#833

Merged
spoorcc merged 4 commits intomainfrom
fix/pre-commit
Oct 18, 2025
Merged

Fix/pre-commit#833
spoorcc merged 4 commits intomainfrom
fix/pre-commit

Conversation

@spoorcc
Copy link
Copy Markdown
Contributor

@spoorcc spoorcc commented Oct 18, 2025

Running the quality task failed since pre-commit calls the tools file-per-file and for some tools this ignores the skip or exclusion list

We should implement #19

Description by Korbit AI

What change is being made?

Update pre-commit configuration to upgrade hooks and refine exclusions:

  • Bump pre-commit-hooks from v5.0.0 to v6.0.0.
  • Exclude generated asciinema-related assets from isort, Black, and codespell checks; and exclude the asciinema player JS file from codespell.
  • Bump gitleaks from v8.16.3 to v8.28.0.

Why are these changes being made?

Keep tooling up-to-date and reduce noise from generated documentation assets during checks; ensure scanning covers newer hook behavior and updated leak-detection rules.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Copy link
Copy Markdown

@korbit-ai korbit-ai 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 completed my review and didn't find any issues... but I did find this beaver.

     .-"""-.__
    /      ' o'\
 ,-;  '.  :   _c
:_."\._ ) ::-"
       ""m "m

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

@spoorcc spoorcc merged commit d45cd1f into main Oct 18, 2025
39 checks passed
@spoorcc spoorcc deleted the fix/pre-commit branch October 18, 2025 18: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