Skip to content

ci: add paths-ignore to skip CI on documentation-only changes - #907

Merged
lizhengfeng101 merged 1 commit into
alibaba:mainfrom
yingjiexu2002:ci/paths-ignore-doc-only
Aug 14, 2026
Merged

ci: add paths-ignore to skip CI on documentation-only changes#907
lizhengfeng101 merged 1 commit into
alibaba:mainfrom
yingjiexu2002:ci/paths-ignore-doc-only

Conversation

@yingjiexu2002

Copy link
Copy Markdown
Collaborator

Description

Adds paths-ignore to the push and pull_request triggers in .github/workflows/ci.yml so changes that no CI step scans no longer spin up the full suite:

paths-ignore:
  - '**.md'
  - '**/LICENSE'
  - '**/.gitignore'

Two small deviations from the list suggested in the issue, both deliberate:

  • docs/** dropped — the repo has no top-level docs/ directory; documentation lives in pages/ and is already covered by **.md
  • LICENSE/.gitignore broadened to **/LICENSE/**/.gitignore — the repo tracks these under extensions/vscode/, pages/ and plugins/ too, and none of them is scanned by any CI step

Kept triggerable, per the issue's analysis: .yml/.yaml (scanned by verify-english-only.go), .github/workflows/**, and action.yml (checked by verify-action-pins.sh).

Doc-oriented checks are unaffected: pages-ci, deploy-pages and translation-sync have their own path-based triggers and keep running for documentation changes.

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that changes existing functionality)
  • Refactoring (no functional changes)
  • Documentation update
  • CI / Build / Tooling

How Has This Been Tested?

  • YAML syntax validated; make check passes
  • This PR itself touches ci.yml, which is not in the ignore list, so its CI run verifies code changes still trigger the workflow
  • The doc-only skip takes effect once merged; a follow-up docs-only PR can confirm it end to end

Checklist

  • My code follows the project's coding style
  • I have performed a self-review of my code
  • I have signed the CLA

Related Issues

closes #905

Add paths-ignore to the push and pull_request triggers so changes that
no CI step scans (markdown files, LICENSE, .gitignore) no longer spin
up the full suite. Source-relevant paths (.yml/.yaml, workflow files,
action.yml) stay triggerable.

Closes alibaba#905
@github-actions

Copy link
Copy Markdown
Contributor

OpenCodeReview: Review complete: 0 finding(s) across 1 selected item(s).

@lizhengfeng101 lizhengfeng101 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@lizhengfeng101
lizhengfeng101 merged commit e770791 into alibaba:main Aug 14, 2026
12 checks passed
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.

ci: add paths-ignore to skip CI on documentation-only changes

2 participants