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

Release/Tag Workflow followup fixes #256

Merged
merged 1 commit into from
Jun 10, 2024

Conversation

egibs
Copy link
Member

@egibs egibs commented Jun 9, 2024

This PR makes a few tweaks/additions:

  1. Uses sed -i instead of sed -i'' (MacOS testing artifact)
  2. Adds version output to the version bump Workflow for quick verification; uses a branch variable (I typo'd the branch name inadvertently which broke the push)
  3. Uses better (and correct) variables for the tag Workflow
  4. Uses GPG signing for tags (which will be needed for gitsign)
  5. Pushes the newly-created tag rather than all tags

Original errors:
CleanShot 2024-06-09 at 17 30 57@2x
CleanShot 2024-06-09 at 17 31 19@2x

@egibs egibs requested a review from tstromberg June 9, 2024 22:36
@egibs egibs force-pushed the release-workflow-followup branch from c921984 to d2cdeea Compare June 9, 2024 22:50
exit 0
else
git config --global tag.gpgsign true
Copy link
Member Author

@egibs egibs Jun 9, 2024

Choose a reason for hiding this comment

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

Copy link
Member Author

Choose a reason for hiding this comment

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

Opened a PR to add git config --global tag.gpgsign true to the Action: chainguard-dev/actions#442

@egibs egibs merged commit ff19192 into chainguard-dev:main Jun 10, 2024
6 checks passed
@egibs egibs deleted the release-workflow-followup branch June 10, 2024 01:53
egibs added a commit to egibs/malcontent that referenced this pull request Aug 5, 2024
egibs added a commit to egibs/malcontent that referenced this pull request Sep 25, 2024
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.

2 participants