skip app-related CI jobs for doc/blog PRs #3704
Merged
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
angiejones
requested changes
Jul 29, 2025
Instead of skipping the entire CI workflow for documentation-only changes, this makes the optimization more granular by: - Adding a 'changes' job to detect which files have changed - Only skipping expensive jobs (rust-format, rust-build-and-test, desktop-lint, bundle-desktop-unsigned) for docs-only PRs - Keeping the workflow running so other important checks (DCO, security scans, etc.) continue to work - Using dorny/paths-filter for reliable file change detection This approach saves significant CI resources for documentation PRs while maintaining all necessary checks for code changes.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
Clean up the workflow by removing the explanatory comments for the conditional job logic.
Pin dorny/paths-filter@v3 to commit SHA de90cc6fb38fc0963ad72b210f1f284cd68cea36 to address the security alert about third-party actions not being pinned.
angiejones
approved these changes
Jul 30, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
This pull request updates the CI workflow configuration to optimize when the workflow is triggered. The most important change is the addition of
paths-ignorerules to exclude documentation-only changes from triggering the CI workflow.Workflow optimization:
.github/workflows/ci.yml: Addedpaths-ignorerules under bothpull_requestandmerge_grouptriggers to prevent the CI workflow from running for changes made exclusively in thedocumentation/**directory.