Skip to content

chore: skip gh CI checks when PR contains only docs and md files#21294

Draft
FollowTheFlo wants to merge 22 commits intodevelopfrom
feature/CXSPA-12325
Draft

chore: skip gh CI checks when PR contains only docs and md files#21294
FollowTheFlo wants to merge 22 commits intodevelopfrom
feature/CXSPA-12325

Conversation

@FollowTheFlo
Copy link
Copy Markdown
Contributor

@FollowTheFlo FollowTheFlo commented Mar 30, 2026

Added a GH action 'DetectChanges'
replaced all workflows that were using

paths-ignore:
      - '**.md'
      - 'docs/**'

by the new DetectChanges action.
It set RUN_CHECKS to true if PR only contains .md or docs/ files
This 'DetectChanges' action is now required to be true prior CI checks.

QA:

Tested with an epic branch (epic/CXSPA-12325) which has the same protection as develop.

  • PR with only md files -> skipped check (CI -Check peerDepenedncies, SonarQube, Unite tests, static code checks)
  • PR with pdf within spartacus/docs folder -> skipped checks
  • Modified a .ts file, push to PR -> all checks running

@FollowTheFlo FollowTheFlo requested a review from a team as a code owner March 30, 2026 03:17
@github-actions github-actions bot marked this pull request as draft March 30, 2026 03:17
@FollowTheFlo FollowTheFlo marked this pull request as ready for review March 30, 2026 03:25
@github-actions github-actions bot marked this pull request as draft March 30, 2026 03:41
@FollowTheFlo FollowTheFlo marked this pull request as ready for review March 30, 2026 18:57
@github-actions
Copy link
Copy Markdown
Contributor

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions bot marked this pull request as draft March 30, 2026 18:57
@FollowTheFlo FollowTheFlo marked this pull request as ready for review March 30, 2026 19:11
@cypress
Copy link
Copy Markdown

cypress bot commented Mar 30, 2026

spartacus    Run #52629

Run Properties:  status check passed Passed #52629  •  git commit 4a60bca2ae ℹ️: Merge a4670b7e174c3d90b1d215e2ad332e097908c096 into a7279d9c65d23725b4aeff3929fc...
Project spartacus
Branch Review feature/CXSPA-12325
Run status status check passed Passed #52629
Run duration 04m 22s
Commit git commit 4a60bca2ae ℹ️: Merge a4670b7e174c3d90b1d215e2ad332e097908c096 into a7279d9c65d23725b4aeff3929fc...
Committer Florent Letendre
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 4
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 101
View all changes introduced in this branch ↗︎

KateChuen
KateChuen previously approved these changes Mar 30, 2026
@github-actions github-actions bot marked this pull request as draft March 30, 2026 19:58
@FollowTheFlo FollowTheFlo marked this pull request as ready for review March 30, 2026 20:21
@github-actions github-actions bot marked this pull request as draft March 30, 2026 21:00
@FollowTheFlo FollowTheFlo marked this pull request as ready for review March 30, 2026 21:24
@github-actions github-actions bot marked this pull request as draft March 31, 2026 12:56
@FollowTheFlo FollowTheFlo marked this pull request as ready for review March 31, 2026 19:36
@github-actions
Copy link
Copy Markdown
Contributor

Merge Checks Failed

Please push a commit to re-trigger the build.
To push an empty commit you can use `git commit --allow-empty -m "Trigger Build"`

@github-actions github-actions bot marked this pull request as draft March 31, 2026 19:37
@FollowTheFlo FollowTheFlo changed the title test: modify md file - do not merge chore: skip gh checks when PR contains only docs and md files Mar 31, 2026
@FollowTheFlo FollowTheFlo changed the title chore: skip gh checks when PR contains only docs and md files chore: skip gh CI checks when PR contains only docs and md files Mar 31, 2026
@FollowTheFlo FollowTheFlo marked this pull request as ready for review April 6, 2026 14:53
@github-actions github-actions bot marked this pull request as draft April 6, 2026 16:05
@FollowTheFlo FollowTheFlo marked this pull request as ready for review April 6, 2026 21:18
@github-actions github-actions bot marked this pull request as draft April 7, 2026 13:33
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.

2 participants