Skip to content

fix(ci): Support docs-only changes without blocking PR merges#41

Merged
kxrm merged 1 commit into
mainfrom
fix/ci-docs-only-support
Jul 2, 2025
Merged

fix(ci): Support docs-only changes without blocking PR merges#41
kxrm merged 1 commit into
mainfrom
fix/ci-docs-only-support

Conversation

@kxrm

@kxrm kxrm commented Jul 2, 2025

Copy link
Copy Markdown
Owner

This PR fixes the CI workflow to properly handle documentation-only changes without blocking PR merges due to missing CI status checks.

Problem

Documentation-only PRs were being blocked because:

  1. The CI workflow used paths-ignore to skip docs/markdown files
  2. Branch protection rules still required "CI Success" status check
  3. When CI doesn't run, there's no status to check → merge blocked

Solution

  • Remove paths-ignore from workflow triggers so CI always runs
  • Add change detection with new check-changes job to identify docs-only PRs
  • Conditional heavy jobs - test/build/security jobs only run for code changes
  • Smart CI Success - validates appropriate jobs based on change type
  • Maintain docs validation - docs job still runs for documentation changes

Benefits

  • 📝 Documentation PRs can merge without waiting for unnecessary code tests
  • 🔧 Code PRs still get full CI validation
  • ⚡ Faster CI for docs-only changes while maintaining protection
  • 🛡️ Branch protection rules work correctly for both change types

Testing

This change enables the documentation PR (#40) to complete CI and merge successfully.

Closes #40 (indirectly by unblocking documentation PRs)

- Remove paths-ignore from workflow triggers to ensure CI always runs
- Add check-changes job to detect documentation-only changes
- Make heavy CI jobs conditional on NOT being docs-only
- Ensure ci-success job passes for both docs-only and code changes
- Fix branch protection rule compatibility for documentation PRs

This resolves the issue where documentation PRs were blocked because
CI didn't run due to paths-ignore, but branch protection still required
the CI Success status check.
@kxrm kxrm merged commit 1e76288 into main Jul 2, 2025
10 checks passed
@kxrm kxrm deleted the fix/ci-docs-only-support branch July 2, 2025 02:26
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