Skip to content

Commit

Permalink
fix: don't check BCR patches on Windows
Browse files Browse the repository at this point in the history
The patches aren't platform-specific so it's redundant to test on windows.
Also our shell one-liner doesn't work on windows.
  • Loading branch information
alexeagle authored Jun 26, 2024
1 parent c09f979 commit d76fcd3
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/bazel.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ jobs:

# See https://github.com/bazel-contrib/publish-to-bcr#including-patches
- name: verify bcr patches
if: matrix.bzlmodEnabled && hashFiles('.bcr/patches/*.patch') != ''
if: matrix.bzlmodEnabled && hashFiles('.bcr/patches/*.patch') != '' && ! startsWith(matrix.os, 'windows')
run: patch --dry-run -p1 < .bcr/patches/*.patch

- name: Test
Expand Down

0 comments on commit d76fcd3

Please sign in to comment.