From c42fde5d078799cc9d8a5e3b241512841db420ea Mon Sep 17 00:00:00 2001 From: Gold856 <117957790+Gold856@users.noreply.github.com> Date: Sun, 3 Aug 2025 14:37:40 -0400 Subject: [PATCH] [ci] Make upstream_utils check error with a more clear message (#8153) Now it links to the README in upstream_utils. --- .github/workflows/upstream-utils.yml | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/.github/workflows/upstream-utils.yml b/.github/workflows/upstream-utils.yml index e4d97ca387..2a10457dad 100644 --- a/.github/workflows/upstream-utils.yml +++ b/.github/workflows/upstream-utils.yml @@ -141,4 +141,11 @@ jobs: - name: Add untracked files to index so they count as changes run: git add -A - name: Check output - run: git --no-pager diff --exit-code HEAD ':!*.bazel' + run: | + set +e + git --no-pager diff --exit-code HEAD ':!*.bazel' + git_exit_code=$? + if test "$git_exit_code" -ne "0"; then + echo "::error ::upstream_utils check failed. This is usually caused by a bad script or the copied files differing from what the script outputs. You can learn more about using upstream_utils to modify thirdparty libraries at https://github.com/wpilibsuite/allwpilib/blob/main/upstream_utils/README.md" + exit $git_exit_code + fi