fix(git): support getting ref in various detached HEAD contexts by neilime · Pull Request #856 · docker/actions-toolkit (original) (raw)

Skip to content

Sign in

Appearance settings

View all features

View all solutions

View all resources

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Sign in

Sign up

Appearance settings

docker / actions-toolkit Public

Additional navigation options

Merged

crazy-max

merged 1 commit into

docker:mainfrom

neilime:fix/git-ref-in-detached-head

Nov 27, 2025

ConversationCommits (1)ChecksFiles changed

Merged

fix(git): support getting ref in various detached HEAD contexts#856

crazy-max

merged 1 commit into

docker:mainfrom

neilime:fix/git-ref-in-detached-head

Conversation

@neilime

@neilime neilime commented

Nov 12, 2025

edited by crazy-max

Loading

Copy link Copy Markdown

Contributor

Support more detached ref formats in Git context.

fixes docker/metadata-action#552

E2E Tests: https://github.com/hoverkraft-tech/test-docker-metadata-action/actions/runs/19305325619

@github-actions github-actions Bot assigned neilime

Nov 12, 2025

@neilime

`[fix(git): support getting ref in various detached HEAD contexts](/docker/actions-toolkit/pull/856/commits/ad7ffdebbe360f9fb771ecd37fb2bcab39e432b3 "fix(git): support getting ref in various detached HEAD contexts

Signed-off-by: Emilien Escalle emilien.escalle@escemi.com") `

[ad7ffde](/docker/actions-toolkit/pull/856/commits/ad7ffdebbe360f9fb771ecd37fb2bcab39e432b3)

Signed-off-by: Emilien Escalle emilien.escalle@escemi.com

@crazy-max

crazy-max commented

Nov 14, 2025

edited

Loading

Copy link Copy Markdown

Member

Thanks I will review this ASAP. For ci issues on macos-15 they are not related, see #852

neilime reacted with thumbs up emoji

crazy-max

crazy-max approved these changes Nov 27, 2025

View reviewed changes

@crazy-max crazy-max left a comment

Copy link Copy Markdown

Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM thanks!

neilime reacted with hooray emoji

@crazy-max crazy-max merged commit 8d01bf1 into docker:main

Nov 27, 2025

97 of 105 checks passed

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.Learn more about bidirectional Unicode characters

[ Show hidden characters]({{ revealButtonHref }})

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

Reviewers

@crazy-max crazy-max crazy-max approved these changes

Assignees

@neilime neilime

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Action fails: Cannot find detached HEAD ref in "HEAD"

2 participants

@neilime @crazy-max