Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ensure that annotated tag content is available #6

Merged
merged 1 commit into from
May 12, 2023

Conversation

seh
Copy link
Owner

@seh seh commented May 11, 2023

As in the similar bisontrails/rules_kustomize#15, when preparing the notes to accompany a release that we issue upon receiving a pushed tag, we try to include both the subject and body of an annotation attached to the motivating Git tag. The actions/checkout GitHub Action does not fetch tag annotations by default, and coercing it to do so (in its current form) would require fetching all of the Git repository's history.

Work around this problem by continuing to fetch the repository history shallowly, but then fetching the repository's tags separately afterward. Doing so makes these annotations available.

When preparing the notes to accompany a release that we issue upon
receiving a pushed tag, we try to include both the subject and body of
an annotation attached to the motivating Git tag. The
"actions/checkout" GitHub Action does not fetch tag annotations by
default, and coercing it to do so (in its current form) would require
fetching all of the Git repository's history.

Work around this problem by continuing to fetch the repository history
shallowly, but then fetching the repository's tags separately
afterward. Doing so makes these annotations available.
@seh seh added the bug Something isn't working label May 11, 2023
@seh seh merged commit 87c0cfe into main May 12, 2023
@seh seh deleted the ensure-annotated-tag-exists-for-release-notes branch August 11, 2023 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants