Skip to content

Commit

Permalink
document minimum Bazel version
Browse files Browse the repository at this point in the history
Change-Id: Id175b8897dd3968dd6bb20908f5783d0a0040412
  • Loading branch information
oliverlee committed Jun 28, 2024
1 parent fac6585 commit 194c817
Show file tree
Hide file tree
Showing 2 changed files with 32 additions and 11 deletions.
38 changes: 31 additions & 7 deletions .github/workflows/check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,21 +47,45 @@ jobs:
strategy:
fail-fast: false
matrix:
include:
- example: default
- example: format-binary
- example: format-config
- example: format-ignore
example:
- default
- format-binary
- format-config
- format-ignore
steps:
- uses: actions/checkout@v4
- shell: bash
run: |
cd example/${{ matrix.example }}
bazel build --config=clang-format-fix
bazel build --config=clang-format
bazel-version:
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
bazel_version:
- latest
- 7.x
- 6.x
- 5.x
steps:
- uses: actions/checkout@v4
- shell: bash
env:
USE_BAZEL_VERSION: ${{ matrix.bazel_version }}
run: |
cd example/default
version=${{ matrix.bazel_version }}
if [[ "$version" == latest || "$version" < 7 ]]; then
sed -i '/enable_bzlmod/d' .bazelrc
fi
bazel build --config=clang-format-fix
all:
runs-on: ubuntu-latest
if: ${{ github.base_ref == 'main' }}
Expand Down
5 changes: 1 addition & 4 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -133,8 +133,5 @@ build:clang-format-base --@bazel_clang_format//:ignore=//:clang-format-ignore

## Requirements

- Bazel ???
- Bazel 5.x
- ClangFormat 14

I'm not sure what the minimum versions are but please let me know if you are
using a version that doesn't work.

0 comments on commit 194c817

Please sign in to comment.