Skip to content

ci: skip pipeline for documentation-only changes#35

Merged
venkatarenduchintala merged 1 commit into
mainfrom
ci/skip-docs-pipeline
Jul 1, 2026
Merged

ci: skip pipeline for documentation-only changes#35
venkatarenduchintala merged 1 commit into
mainfrom
ci/skip-docs-pipeline

Conversation

@venkatarenduchintala

Copy link
Copy Markdown
Owner

What

Stops the validate pipeline from building for documentation-only work — it is not worth a full devcontainer build + Neovim/treesitter suite for docs.

Changes to .github/workflows/validate.yml

  • on.push now uses branches-ignore: ["docs/**"] — pushes to docs/* branches no longer trigger the workflow at all.

  • Job if now skips:

    • PRs opened from a docs/* branch (github.head_ref), and
    • pushes whose tip commit is docs: / docs(scope):.

    The previous condition only matched bare docs: / chore: and always ran on any pull request, so scoped docs commits (e.g. docs(readme):) and docs PRs still built. chore: / chore(scope): remain skipped as before.

Expression semantics

On pull_request events head_commit is null, so only the head_ref check applies; on push events head_ref is empty, so only the message checks apply. Pushes to docs/** are filtered before the job by on.push.

Validated: YAML parses (python -c yaml.safe_load) and yamllint -d relaxed is clean.

Restrict the validate workflow so it does not build for docs work:
- on.push uses branches-ignore: "docs/**" — pushes to docs branches never
  trigger the workflow.
- the job `if` skips PRs opened from a docs/* branch (github.head_ref) and
  pushes whose tip commit is docs:/docs(scope):. The previous check only
  matched bare docs:/chore: and missed the scoped conventional-commit form
  (e.g. docs(readme):), so docs PRs still built.
@venkatarenduchintala venkatarenduchintala merged commit dda9ec6 into main Jul 1, 2026
2 checks passed
@venkatarenduchintala venkatarenduchintala deleted the ci/skip-docs-pipeline branch July 1, 2026 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant