ci: Make docs-only check a no-op in the merge queue (#22576)
Marshall Bowers
created
This PR makes the docs-only check a no-op that defaults to `false` when
running in the merge queue.
I noticed that the current check did not work properly in the merge
queue, resulting in it always assuming a change was docs-only and not
running the requisite CI jobs.
Release Notes:
- N/A
@@ -35,7 +35,12 @@ jobs:
- name: Check for non-docs changes
id: check_changes
run: |
- if git diff --name-only ${{ github.event.pull_request.base.sha }} ${{ github.sha }} | grep -qvE '^docs/'; then
+ if [ "${{ github.event_name }}" == "merge_group" ]; then
+ # When we're running in a merge queue, never assume that the changes
+ # are docs-only, as there could be other PRs in the group that
+ # contain non-docs changes.
+ echo "docs_only=false" >> $GITHUB_OUTPUT
+ elif git diff --name-only ${{ github.event.pull_request.base.sha }} ${{ github.sha }} | grep -qvE '^docs/'; then
echo "docs_only=false" >> $GITHUB_OUTPUT
else
echo "docs_only=true" >> $GITHUB_OUTPUT