Sets heavy GitHub CI workflows to not trigger on text documentation-only changes. #2417
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
github actions
dgaliffiAMD
approved these changes
Dec 19, 2025
neon60
reviewed
Jan 5, 2026
Specifically, sets azure-ci-dispatcher.yml and therock-ci.yml to not trigger when the change consists entirely of .txt, .md, or .rtf files.
Most notably, any changes in the 'docs' folder, or the 'docs' folder in any project, are now excluded.
dsclear-amd
force-pushed
the
user/dsclear/shorten_pr_checks_for_docs_changes_commitable
branch
from
8a4520b to
d3762e4
Compare
This workflow should now correctly skip docs-only changes, even on pushes.
…hanges_commitable
This should eliminate unnecessary testing, and speed up documentation development.
neon60
reviewed
Jan 8, 2026
neon60
approved these changes
Jan 8, 2026
adeljo-amd
approved these changes
Jan 8, 2026
Specifically, fixes the old incorrect paths like '**.txt' to be of the style '**/*.txt'.
Specifically, changes them from the incorrect '**.txt' style to '**/*.txt'.
geomin12
requested changes
Jan 9, 2026
Contributor
geomin12
left a comment
geomin12
left a comment
There was a problem hiding this comment.
looks fine, although this will break PRs for therock-ci.yml, as TheRock CI is required to run
please add here for TheRock:
… configure script.
Contributor
|
Is it safe to assume changes to *.txt are non-functional (and don't require testing?). For example, maybe changes to a CMakeLists.txt? |
Contributor
Author
Excellent point, I didn't consider CMakeLists.txt. I'll remove ignoring .txt files in favour of just listing docs-specific .txt files we know to ignore. |
Specifically, instead of excluding all .txt files (which would catch things like CMakeLists.txt), now only specific .txt files are excluded.
j-stephan
approved these changes
Jan 12, 2026
…hanges_commitable
This is because using both paths and paths-ignore at the same time is not allowed.
geomin12
approved these changes
Jan 12, 2026
dsclear-amd
deleted the
user/dsclear/shorten_pr_checks_for_docs_changes_commitable
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation
Currently, full installation and unit test runs are conducted even for changes of only documentation files; this greatly slows down docs development work.
Technical Details
This change modifies two of the largest workflows, azure-ci-dispatcher.yml and therock-ci.yml, as well as many rocprofiler workflows, to not trigger on PRs or pushes that consist entirely of documentation changes.
JIRA ID
AIROCDOC-78, AIROCDOC-424, AIROCDOC-425
Test Plan
Test Result
The test_branch_bin_feeder PR caused the modified workflows to still trigger, as expected. The test_branch_text_feeder PR caused the modified workflows to be skipped, as desired.