Skip to content

Commit 1a00a81

Browse files
committed
Fix bug in update getting started guide workflow
This change fixes a bug in the update getting started guide workflow when assertions are disabled. This change also fixes a bug where '--verbose' executed git diff command in interactive mode. Signed-off-by: Austin Vazquez <[email protected]>
1 parent 184d564 commit 1a00a81

File tree

2 files changed

+9
-9
lines changed

2 files changed

+9
-9
lines changed

.github/workflows/update-getting-started-guide.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,21 +23,15 @@ jobs:
2323
contents: read
2424
pull-requests: read
2525

26-
env:
27-
RELEASE_TAG: ''
28-
2926
steps:
3027
- uses: actions/checkout@v4
3128
with:
3229
sparse-checkout: |
3330
docs/getting-started.md
3431
scripts/update-getting-started-guide-version.sh
3532
36-
- name: Mock release tag on pull request
37-
run: echo "RELEASE_TAG=v0.0.0-${{ github.event.pull_request.number }}" >> $GITHUB_ENV
38-
3933
- name: Test update getting started version
40-
run: bash scripts/update-getting-started-guide-version.sh --assert ${{ env.RELEASE_TAG }}
34+
run: bash scripts/update-getting-started-guide-version.sh --assert v0.0.0-${{ github.event.pull_request.number }}
4135

4236
update-version:
4337
if: github.event_name == 'release'

scripts/update-getting-started-guide-version.sh

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -69,5 +69,11 @@ assert_diff() {
6969

7070
sed -i -E "s/version=\"([0-9]+\.){2}[0-9]+\"/version=\"${VERSION}\"/" docs/getting-started.md
7171

72-
[ $VERBOSE = true ] && git diff
73-
[ $ASSERT = true ] && assert_diff
72+
if [ $VERBOSE = true ]; then
73+
git --no-pager diff
74+
fi
75+
76+
if [ $ASSERT = true ]; then
77+
assert_diff
78+
fi
79+

0 commit comments

Comments
 (0)