Skip to content

Commit 137b0a5

Browse files
authored
Bump manifest in presubmit CI. (#791)
It is still possible to manually launch the pipeline without the bump. This is following on from #770. Further simplification will be possible. #782 has become too large, this is the minimal change to make the presubmit CI behave like the nightlies.
1 parent 94f09fe commit 137b0a5

File tree

1 file changed

+7
-3
lines changed

1 file changed

+7
-3
lines changed

.github/workflows/ci.yaml

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,8 @@ on:
2121
BUMP_MANIFEST:
2222
type: boolean
2323
description: Bump git repos in manifest.yaml to head of tree?
24-
default: false
25-
required: false
24+
default: true
25+
required: true
2626
MERGE_BUMPED_MANIFEST:
2727
type: boolean
2828
description: "(used if BUMP_MANIFEST=true) If true: attempt to PR/merge manifest branch"
@@ -88,7 +88,11 @@ jobs:
8888
id: manifest-branch
8989
shell: bash -x -e {0}
9090
run: |
91-
BUMP_MANIFEST=${{ github.event_name == 'schedule' || inputs.BUMP_MANIFEST || 'false' }}
91+
if [[ "${{ github.event_name }}" == "workflow_dispatch" ]]; then
92+
BUMP_MANIFEST="${{ inputs.BUMP_MANIFEST }}"
93+
else
94+
BUMP_MANIFEST="true"
95+
fi
9296
MERGE_BUMPED_MANIFEST=${{ github.event_name == 'schedule' || inputs.MERGE_BUMPED_MANIFEST || 'false' }}
9397
# Prepend nightly manifest branch with "z" to make it appear at the end
9498
if [[ "$BUMP_MANIFEST" == "true" ]]; then

0 commit comments

Comments
 (0)