Skip to content

ci: skip workflows on docs-only changes - #30

Merged
tkhquang merged 1 commit into
mainfrom
ci/skip-workflows-on-docs
Mar 22, 2026
Merged

ci: skip workflows on docs-only changes#30
tkhquang merged 1 commit into
mainfrom
ci/skip-workflows-on-docs

Conversation

@tkhquang

@tkhquang tkhquang commented Mar 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Add paths filter to pr-check.yml and coverage-pages.yml so they only trigger on source, build config, or workflow file changes
  • Docs-only PRs and pushes (README, .md files, etc.) no longer run the full build/test/coverage pipeline
  • Manual workflow_dispatch is preserved for on-demand runs

Summary by CodeRabbit

  • Chores
    • Optimized CI/CD workflow triggers to run only when relevant files are modified, improving build efficiency.

@tkhquang tkhquang self-assigned this Mar 22, 2026
@coderabbitai

coderabbitai Bot commented Mar 22, 2026

Copy link
Copy Markdown
📝 Walkthrough

Walkthrough

Added paths filters to two GitHub Actions workflows (coverage-pages.yml and pr-check.yml), restricting execution to changes affecting CMake configuration, external dependencies, include files, source code, tests, or the workflow files themselves.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/coverage-pages.yml, .github/workflows/pr-check.yml
Added paths filter to trigger conditions, limiting workflow runs to only when changes are made to CMakeLists.txt, CMakePresets.json, external/**, include/**, src/**, tests/**, or the workflow file itself.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: adding path filters to skip CI workflows when only documentation files are modified.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🧹 Nitpick comments (1)
.github/workflows/coverage-pages.yml (1)

6-13: Broaden workflow-file matching to avoid trigger drift.

This filter only includes one workflow file. If the intent is to run on workflow changes generally, use a broader pattern.

♻️ Suggested change
-      - '.github/workflows/coverage-pages.yml'
+      - '.github/workflows/**'
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In @.github/workflows/coverage-pages.yml around lines 6 - 13, The workflow's
paths filter explicitly lists only '.github/workflows/coverage-pages.yml', which
will miss other workflow file changes and cause trigger drift; update the paths
block (the list containing 'CMakeLists.txt', 'CMakePresets.json', ...,
'.github/workflows/coverage-pages.yml') to use a broader pattern such as
'.github/workflows/**' or include the appropriate wildcard(s) so changes to any
workflow file will trigger this workflow instead of only coverage-pages.yml.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In @.github/workflows/coverage-pages.yml:
- Around line 6-13: The workflow's paths filter explicitly lists only
'.github/workflows/coverage-pages.yml', which will miss other workflow file
changes and cause trigger drift; update the paths block (the list containing
'CMakeLists.txt', 'CMakePresets.json', ...,
'.github/workflows/coverage-pages.yml') to use a broader pattern such as
'.github/workflows/**' or include the appropriate wildcard(s) so changes to any
workflow file will trigger this workflow instead of only coverage-pages.yml.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c887dfe4-57e6-4467-afdf-6f420692b9a2

📥 Commits

Reviewing files that changed from the base of the PR and between a57bbfb and 91e6f7b.

📒 Files selected for processing (2)
  • .github/workflows/coverage-pages.yml
  • .github/workflows/pr-check.yml

@tkhquang
tkhquang merged commit 344eaa9 into main Mar 22, 2026
2 checks passed
@tkhquang
tkhquang deleted the ci/skip-workflows-on-docs branch March 23, 2026 21:21
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