Skip to content

Commit

Permalink
make the PR title and message clear that the smoke test was run and i…
Browse files Browse the repository at this point in the history
…t did pass. (#239)
  • Loading branch information
jmazzitelli authored Dec 11, 2023
1 parent e197b95 commit 5393acd
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -222,8 +222,8 @@ jobs:
git push origin $(git rev-parse HEAD):refs/heads/$BUILD_TAG
if hack/smoke-test-release-branch.sh --release-branch $BUILD_TAG; then
prtitle="Prepare for next version"
prmsg="Please merge to update version numbers and prepare for release $NEXT_VERSION"
prtitle="Prepare for next version (smoke test passed)"
prmsg="The smoke test has passed. Please merge to update version numbers and prepare for release $NEXT_VERSION"
else
prtitle="[DO NOT MERGE YET] Prepare for next version"
prmsg="DO NOT MERGE YET! The smoke test failed. Please fix the problem before merging this PR which updates version numbers and prepares for release $NEXT_VERSION"
Expand All @@ -238,8 +238,8 @@ jobs:
git push origin $(git rev-parse HEAD):refs/heads/$BUILD_TAG
if hack/smoke-test-release-branch.sh --release-branch $BUILD_TAG; then
prtitle="Prepare for next version"
prmsg="Please merge to update version numbers and prepare for release $NEXT_VERSION"
prtitle="Prepare for next version (smoke test passed)"
prmsg="The smoke test has passed. Please merge to update version numbers and prepare for release $NEXT_VERSION"
else
prtitle="[DO NOT MERGE YET] Prepare for next version"
prmsg="DO NOT MERGE YET! The smoke test failed. Please fix the problem before merging this PR which updates version numbers and prepares for release $NEXT_VERSION"
Expand Down

0 comments on commit 5393acd

Please sign in to comment.