Skip to content

ci: skip docs validation for dependabot PRs#391

Merged
cusma merged 4 commits intomainfrom
ci/dependabot-skips-docs-check
Feb 18, 2026
Merged

ci: skip docs validation for dependabot PRs#391
cusma merged 4 commits intomainfrom
ci/dependabot-skips-docs-check

Conversation

@cusma
Copy link
Collaborator

@cusma cusma commented Feb 17, 2026

No description provided.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the docs validation workflow to avoid running docs checks for Dependabot-authored pull requests, keeping required checks green while reducing CI work.

Changes:

  • Add a Dependabot PR detection step that sets an is_dependabot output.
  • Skip checkout and docs change detection when the PR is identified as Dependabot.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (2)

.github/workflows/docs-ci.yaml:62

  • The condition at line 62 will fail for workflow_dispatch and workflow_call events because github.event.pull_request will be null/undefined in those contexts. This step needs to check github.event_name first to avoid evaluation errors. The condition should be: github.event_name != 'pull_request' || github.event.pull_request.user.login != 'dependabot[bot]'
              - 'docs/**'

.github/workflows/docs-ci.yaml:39

  • The new dependabot skip message step will run but then the workflow stops at line 41 because no checkout occurs for dependabot PRs. This means the job succeeds early. However, consider whether this is the intended behavior or if you want to explicitly exit with success. The current implementation relies on subsequent steps being skipped, which works but may not be immediately clear. Consider adding a comment explaining this flow or using a job-level condition instead.
      - name: Checkout source code (PR)
        if: github.event_name == 'pull_request'
        uses: actions/checkout@v6

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@cusma cusma merged commit f3091c6 into main Feb 18, 2026
8 checks passed
@cusma cusma deleted the ci/dependabot-skips-docs-check branch February 18, 2026 05:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments