Commit 14d3924
committed
tools: fix test426 updater
The previous version produces a commit that does pass the linter
because of a too-long commit title.
Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com>
PR-URL: #63271
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>1 parent 9f319a7 commit 14d3924
1 file changed
Lines changed: 6 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
@@ -17,10 +17,10 @@ fi | |||
| 17 | 17 | TARBALL_URL=$(curl -fsIo /dev/null -w '%header{Location}' https://github.com/tc39/source-map-tests/archive/HEAD.tar.gz) | |
| 18 | 18 | SHA=$(basename "$TARBALL_URL") | |
| 19 | 19 | ||
| 20 | - if [ "$CURRENT_SHA" = "$SHA" ]; then | ||
| 21 | - echo "Already up-to-date" | ||
| 22 | - exit 0 | ||
| 23 | - fi | ||
| 20 | + # shellcheck disable=SC1091 | ||
| 21 | + . "$BASE_DIR/tools/dep_updaters/utils.sh" | ||
| 22 | + | ||
| 23 | + compare_dependency_version "test426-fixtures" "$CURRENT_SHA" "$SHA" | ||
| 24 | 24 | ||
| 25 | 25 | rm -rf "$TARGET_DIR" | |
| 26 | 26 | mkdir "$TARGET_DIR" | |
@@ -32,4 +32,5 @@ mv "$TMP_FILE" "$README" | |||
| 32 | 32 | ||
| 33 | 33 | # The last line of the script should always print the new version, | |
| 34 | 34 | # as we need to add it to $GITHUB_ENV variable. | |
| 35 | - echo "NEW_VERSION=$SHA" | ||
| 35 | + NEW_VERSION=$(echo "$SHA" | head -c 39) | ||
| 36 | + echo "NEW_VERSION=$NEW_VERSION" | ||
0 commit comments