Skip to content

Commit 14d3924

Browse files
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

File tree

tools/dep_updaters/update-test426-fixtures.sh

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -17,10 +17,10 @@ fi
1717
TARBALL_URL=$(curl -fsIo /dev/null -w '%header{Location}' https://github.com/tc39/source-map-tests/archive/HEAD.tar.gz)
1818
SHA=$(basename "$TARBALL_URL")
1919

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"
2424

2525
rm -rf "$TARGET_DIR"
2626
mkdir "$TARGET_DIR"
@@ -32,4 +32,5 @@ mv "$TMP_FILE" "$README"
3232

3333
# The last line of the script should always print the new version,
3434
# 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

Comments
 (0)