Skip to content

Checkout the proper branch on CI to replicate local behavior#6225

Merged
sebastianbergmann merged 1 commit intosebastianbergmann:mainfrom
soyuka:fix/ci
Jun 4, 2025
Merged

Checkout the proper branch on CI to replicate local behavior#6225
sebastianbergmann merged 1 commit intosebastianbergmann:mainfrom
soyuka:fix/ci

Conversation

@soyuka
Copy link
Copy Markdown
Contributor

@soyuka soyuka commented Jun 3, 2025

Because actions/checkout uses a detached HEAD, the code at InfrastructureInformationProvider (symbolic-ref --short HEAD) doesn't get the correct information. This patch makes sure we checkout the correct branch to replicate our local usage.

I've tested this at https://github.com/soyuka/phpunit/actions/runs/15427391478/job/43417670031.

@sebastianbergmann
Copy link
Copy Markdown
Owner

We also need this change for the code-coverage job. Thank you for working on this!

@soyuka
Copy link
Copy Markdown
Contributor Author

soyuka commented Jun 4, 2025

interesting code coverage was green on my fork's CI. I added the patch on the coverage as well.

@sebastianbergmann sebastianbergmann merged commit e76bbda into sebastianbergmann:main Jun 4, 2025
22 of 23 checks passed
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Apr 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants