Skip to content

ignore uv.lock and bump pre-commit hooks #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
deependujha wants to merge 4 commits into
base: main
Choose a base branch
from
Open

ignore uv.lock and bump pre-commit hooks #6

deependujha wants to merge 4 commits into from

Conversation

@deependujha
Copy link

@deependujha deependujha commented Dec 21, 2025

  • Explicitly ignore uv.lock since this repository is a library
  • Update pre-commit-hooks to v6.0.0 to address deprecations warning

@Borda
Copy link

Borda commented Jan 21, 2026

seems that some tests require/expect secrets that are not shared with forked PRs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@justusschock justusschock justusschock approved these changes

@tchaton tchaton Awaiting requested review from tchaton tchaton is a code owner

@ethanwharris ethanwharris Awaiting requested review from ethanwharris ethanwharris is a code owner

@KaelanDt KaelanDt Awaiting requested review from KaelanDt KaelanDt is a code owner

+1 more reviewer

@Borda Borda Borda approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@deependujha @Borda @justusschock