Skip to content

Commit

Permalink
fix: BCR presubmit should test the release path
Browse files Browse the repository at this point in the history
Because bzlmod users get a module that doesn't have a rules_rust dep
  • Loading branch information
alexeagle committed Apr 25, 2024
1 parent 734070b commit 45ce7d6
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .bcr/presubmit.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
bcr_test_module:
module_path: "e2e/smoke"
module_path: "e2e/use_release"
matrix:
bazel: ["7.x", "6.x"]
platform: ["debian10", "macos", "ubuntu2004", "windows"]
Expand Down
4 changes: 3 additions & 1 deletion e2e/use_release/minimal_download_test.sh
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ fi

# This test references pre-built artifacts from a prior release.
# Will need to bump this version in the future when there are breaking changes.
export RULES_PY_RELEASE_VERSION=0.7.0
export RULES_PY_RELEASE_VERSION=0.7.1

#############
# Test bzlmod
Expand All @@ -38,6 +38,8 @@ then
exit 1
fi

bazel "--output_base=$OUTPUT_BASE" test --enable_bzlmod //...

#############
# Test WORKSPACE
OUTPUT_BASE=$(mktemp -d)
Expand Down

0 comments on commit 45ce7d6

Please sign in to comment.