Skip to content

Fix Internal Tests paths filter checkout#5295

Open
clockwork-labs-bot wants to merge 1 commit into
masterfrom
bot/fix-internal-tests-paths-filter
Open

Fix Internal Tests paths filter checkout#5295
clockwork-labs-bot wants to merge 1 commit into
masterfrom
bot/fix-internal-tests-paths-filter

Conversation

@clockwork-labs-bot

@clockwork-labs-bot clockwork-labs-bot commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

What changed

Adds an explicit checkout step before dorny/paths-filter in the Internal Tests workflow.

Why

dorny/paths-filter@v3 needs a git working tree for push events. The Internal Tests workflow ran it before any checkout, so every push run on master failed immediately in Detect non-docs changes with:

fatal: not a git repository (or any of the parent directories): .git

This only showed up consistently on master because those runs are push events. On pull_request events, dorny/paths-filter can use the GitHub pull request files API with the PR number, so it does not need a local checkout for the same file detection path.

Adding checkout gives the action a repository when it handles push events, while leaving PR behavior unchanged.

Testing

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