Skip to content

Commit

Permalink
ci(pre-commit): Autoupdate (#474)
Browse files Browse the repository at this point in the history
The pre-commit hooks are quite old in this repo. This PR run `pre-commit
autoupdate`. Since the hooks are not enfored by CI, buildifier is
already broken. I will create a follow up PR similar to
bazel-contrib/bazel-lib#1002 that adds it to CI
and fix all errors.
  • Loading branch information
hofbi authored Dec 13, 2024
1 parent e064327 commit 0b3c298
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 4 deletions.
6 changes: 3 additions & 3 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,18 @@ default_stages: [commit]
repos:
# Check formatting and lint for starlark code
- repo: https://github.com/keith/pre-commit-buildifier
rev: 4.0.1.1
rev: 7.3.1.1
hooks:
- id: buildifier
- id: buildifier-lint
# Enforce that commit messages allow for later changelog generation
- repo: https://github.com/commitizen-tools/commitizen
rev: v2.18.0
rev: v4.1.0
hooks:
# Requires that commitizen is already installed
- id: commitizen
stages: [commit-msg]
- repo: https://github.com/pre-commit/mirrors-prettier
rev: "v2.4.0"
rev: "v3.1.0"
hooks:
- id: prettier
1 change: 0 additions & 1 deletion e2e/repository-rule-deps/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ The `:test` test case imports two modules from `@myrepo` ("flat" using `imports=

The second test case `:all_direct` skips the `repository_rule` to demonstrate that the `test.py` itself is correctly implemented and can actually pass when the bug is not present.


```console
$ bazel test ...
...
Expand Down

0 comments on commit 0b3c298

Please sign in to comment.