Skip to content

fix: add checkout step back to mergeability check#15783

Open
iCiaran wants to merge 2 commits intowithastro:mainfrom
iCiaran:check-merge-checkout
Open

fix: add checkout step back to mergeability check#15783
iCiaran wants to merge 2 commits intowithastro:mainfrom
iCiaran:check-merge-checkout

Conversation

@iCiaran
Copy link

@iCiaran iCiaran commented Mar 5, 2026

Changes

The checkout step was removed in #14454 to force the changed-files action to use the github api to identify changed files, since a local git diff is not reliable from a forked repository.

The checkout is however required before the check so the markdown files can be examined. See recent runs of the workflow, which error in the " Check if any changesets contain minor or major changes" step since the markdown files can't be found for example

Checking for changesets marked as minor or major
cat: .changeset/harden-node-server-defaults.md: No such file or directory

Testing

Docs

Pull request CI step, no docs required.

@github-actions github-actions bot added the 🚨 action Modifies GitHub Actions label Mar 5, 2026
@changeset-bot
Copy link

changeset-bot bot commented Mar 5, 2026

⚠️ No Changeset found

Latest commit: bdbcd8c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🚨 action Modifies GitHub Actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant