Skip to content

Don't skip CI on md files#2073

Open
Dreamsorcerer wants to merge 1 commit into
mainfrom
Dreamsorcerer-patch-2
Open

Don't skip CI on md files#2073
Dreamsorcerer wants to merge 1 commit into
mainfrom
Dreamsorcerer-patch-2

Conversation

@Dreamsorcerer
Copy link
Copy Markdown
Collaborator

At the very least, we need to run codeblocks and get the ci-complete status to pass when md files are changed.

I think it's easiest currently to just run the full workflow, as it doesn't take that long. Maybe in future we'll adapt this to only run specific steps depending on the files changed.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 13, 2026

Greptile Summary

This PR removes the paths-ignore: ['**.md'] filters from both the push and pull_request workflow triggers in .github/workflows/ci.yml. This ensures the full CI workflow (including md-babel codeblock validation and the ci-complete branch-protection gate) runs on markdown-only changes. No issues were found.

Confidence Score: 5/5

Safe to merge — a four-line deletion with a clear, correct intent and no functional regressions.

The change is a minimal, intentional removal of path filters that were preventing the md-babel and ci-complete jobs from running on markdown-only PRs. Nothing in the remaining workflow logic is altered.

No files require special attention — only .github/workflows/ci.yml is touched.

Important Files Changed

Filename Overview
.github/workflows/ci.yml Removes paths-ignore: ['**.md'] from both the push and pull_request triggers so the full CI workflow runs on markdown-only changes.

Reviews (1): Last reviewed commit: "Don't skip CI on md files" | Re-trigger Greptile

@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

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