From 3e3c83f0ca6c73cd6f2513640b7ed04d911fbb25 Mon Sep 17 00:00:00 2001 From: Mike Kruskal Date: Fri, 1 Sep 2023 15:54:27 -0700 Subject: [PATCH] Clarify staleness tests - They will now output the regen command on failure - On fork presubmits, they won't be named by branch PiperOrigin-RevId: 562068279 --- .github/workflows/staleness_check.yml | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/.github/workflows/staleness_check.yml b/.github/workflows/staleness_check.yml index b8dd978da7..04a1a1c3d8 100644 --- a/.github/workflows/staleness_check.yml +++ b/.github/workflows/staleness_check.yml @@ -21,14 +21,14 @@ jobs: branch: [main, 22.x, 23.x, 24.x] os: [{ name: Linux, value: ubuntu-latest}] - name: Test staleness ${{ matrix.os.name }} ${{ matrix.branch}} + name: Test staleness ${{ matrix.os.name }} ${{ inputs.safe-checkout && 'PR' || matrix.branch }} runs-on: ${{ matrix.os.value }} if: ${{ github.event.repository.full_name == 'protocolbuffers/protobuf' }} steps: - - name: Checkout ${{ matrix.branch }} + - name: Checkout ${{ inputs.safe-checkout && 'PR' || matrix.branch }} uses: actions/checkout@ac593985615ec2ede58e132d2e21d2b1cbd6127c # v3.3.0 with: - ref: ${{ inputs.safe-checkout || matrix.branch}} + ref: ${{ inputs.safe-checkout || matrix.branch }} - name: Mark runs associated with commits if: ${{ github.event_name != 'schedule' && github.event_name != 'workflow_dispatch' }} @@ -51,7 +51,8 @@ jobs: # commit. run: | if [[ -z $COMMIT_TRIGGERED_RUN || -z $MAIN_RUN ]]; then - bazel query 'attr(tags, "staleness_test", //...)' | xargs bazel test $BAZEL_FLAGS + bazel query 'attr(tags, "staleness_test", //...)' | xargs bazel test $BAZEL_FLAGS || \ + echo "Please run ./regenerate_stale_files.sh to regenerate stale files" else bazel query 'attr(tags, "staleness_test", //...)' fi