Skip to content
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

ci: Check pre-commit hooks in CI #922

Merged
merged 4 commits into from
Feb 3, 2025
Merged

ci: Check pre-commit hooks in CI #922

merged 4 commits into from
Feb 3, 2025
+22 −2

Conversation

hofbi
Copy link
Contributor

@hofbi hofbi commented Jan 20, 2025

Follow up of #913 making sure all pre-commit hooks stay green.

@hofbi hofbi requested a review from meteorcloudy as a code owner January 20, 2025 09:48
@hofbi hofbi requested review from aiuto and cgrindel as code owners January 20, 2025 09:56
@hofbi hofbi mentioned this pull request Jan 20, 2025
Merged
@hofbi
Copy link
Contributor Author

hofbi commented Feb 3, 2025

@aiuto a kind reminder for a review

@cgrindel
Copy link
Collaborator

cgrindel commented Feb 3, 2025

Thanks for the other cleanup changes, as well.

@aiuto aiuto merged commit 1772ebd into bazelbuild:main Feb 3, 2025
3 checks passed
@hofbi hofbi deleted the pre-commit-gh-action branch February 3, 2025 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cgrindel cgrindel cgrindel approved these changes

@aiuto aiuto aiuto approved these changes

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

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
@hofbi @cgrindel @aiuto