Skip to content

Commit

Permalink
Prevent duplicate queries during hashing (#203)
Browse files Browse the repository at this point in the history
* Prevent duplicate queries during hashing

Currently duplicate queries could fire with this logic

* pin version
  • Loading branch information
tinder-maxwellelliott authored Jan 30, 2024
1 parent c47d226 commit 2efca5f
Show file tree
Hide file tree
Showing 9 changed files with 657 additions and 57 deletions.
1 change: 1 addition & 0 deletions .bazelversion
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
6.4.0
10 changes: 5 additions & 5 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,7 @@ jobs:
run: go install github.com/bazelbuild/bazelisk@latest && export PATH=$PATH:$(go env GOPATH)/bin
- uses: actions/checkout@v3
- name: Run bazel-diff tests
run: USE_BAZEL_VERSION=latest ~/go/bin/bazelisk test //cli/...
run: ~/go/bin/bazelisk test //cli/...
- name: archive testlogs
if: always()
run: cp -R $(~/go/bin/bazelisk info bazel-testlogs) ./test-logs; (cd test-logs; zip -r -X ../test-logs.zip .)
Expand Down Expand Up @@ -55,7 +55,7 @@ jobs:
with:
fetch-depth: 0
- name: Run bazel-diff example script
run: USE_BAZEL_VERSION=latest ./bazel-diff-example.sh "$GITHUB_WORKSPACE" ~/go/bin/bazelisk $(git rev-parse HEAD~1) $(git rev-parse HEAD)
run: ./bazel-diff-example.sh "$GITHUB_WORKSPACE" ~/go/bin/bazelisk $(git rev-parse HEAD~1) $(git rev-parse HEAD)
test-jre11:
runs-on: ubuntu-latest
steps:
Expand All @@ -73,7 +73,7 @@ jobs:
run: go install github.com/bazelbuild/bazelisk@latest && export PATH=$PATH:$(go env GOPATH)/bin
- uses: actions/checkout@v3
- name: Run bazel-diff tests
run: USE_BAZEL_VERSION=latest ~/go/bin/bazelisk coverage --combined_report=lcov //cli/...
run: ~/go/bin/bazelisk coverage --combined_report=lcov //cli/...
- name: archive testlogs
if: always()
run: cp -R $(~/go/bin/bazelisk info bazel-testlogs) ./test-logs; (cd test-logs; zip -r -X ../test-logs.zip .)
Expand Down Expand Up @@ -103,7 +103,7 @@ jobs:
with:
fetch-depth: 0
- name: Run bazel-diff example script
run: USE_BAZEL_VERSION=latest ./bazel-diff-example.sh "$GITHUB_WORKSPACE" ~/go/bin/bazelisk $(git rev-parse HEAD~1) $(git rev-parse HEAD)
run: ./bazel-diff-example.sh "$GITHUB_WORKSPACE" ~/go/bin/bazelisk $(git rev-parse HEAD~1) $(git rev-parse HEAD)
deploy:
needs: [test-jre8, test-jre11]
runs-on: ubuntu-latest
Expand All @@ -126,7 +126,7 @@ jobs:
run: go install github.com/bazelbuild/bazelisk@latest && export PATH=$PATH:$(go env GOPATH)/bin
- uses: actions/checkout@v3
- name: Build deployable JAR
run: USE_BAZEL_VERSION=latest ~/go/bin/bazelisk build //cli:bazel-diff_deploy.jar
run: ~/go/bin/bazelisk build //cli:bazel-diff_deploy.jar
- uses: actions/upload-artifact@v3
with:
name: bazel-diff_deploy.jar
Expand Down
6 changes: 6 additions & 0 deletions MODULE.bazel
Original file line number Diff line number Diff line change
@@ -0,0 +1,6 @@
###############################################################################
# Bazel now uses Bzlmod by default to manage external dependencies.
# Please consider migrating your external dependencies from WORKSPACE to MODULE.bazel.
#
# For more details, please check https://github.com/bazelbuild/bazel/issues/18958
###############################################################################
Loading

0 comments on commit 2efca5f

Please sign in to comment.