Skip to content

Release version 2.5.0#366

Merged
rafaelfranca merged 2 commits intomainfrom
release-v2.5.0
Mar 4, 2022
Merged

Release version 2.5.0#366
rafaelfranca merged 2 commits intomainfrom
release-v2.5.0

Conversation

@tomstuart
Copy link
Contributor

We need to bump the version number to get Shipit to publish the gem. The tag and release already exist, and therefore won’t point to this commit, but I don’t think that will cause any real problems.

Copy link
Contributor

@volmer volmer left a comment

Choose a reason for hiding this comment

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

Thank you!! ❤️

@sambostock
Copy link
Contributor

sambostock commented Mar 3, 2022

Can we/should we move the tag after merging? Seems weird that the 2.5.0 tag would point to code with a different version.

@tomstuart
Copy link
Contributor Author

Can we/should we move the tag after merging? Seems weird that the 2.5.0 tag would point to code with a different version.

We can, and (in my opinion) we should, but I don’t know if that’ll cause problems anywhere. I’ll wait for @rafaelfranca’s review.

@rafaelfranca rafaelfranca merged commit 7c0b4f2 into main Mar 4, 2022
@rafaelfranca rafaelfranca deleted the release-v2.5.0 branch March 4, 2022 16:13
@shopify-shipit shopify-shipit bot temporarily deployed to rubygems March 4, 2022 16:15 Inactive
@tomstuart
Copy link
Contributor Author

Can we/should we move the tag after merging? Seems weird that the 2.5.0 tag would point to code with a different version.

We can, and (in my opinion) we should

(We did.)

@paracycle
Copy link
Member

paracycle commented Mar 7, 2022

Btw, you don't need to manually tag repos when doing a release, Shipit creates and pushes a tag when it is deploying the gem to Rubygems.

@rafaelfranca
Copy link
Member

It doesn't sign the tag though, so I always create manually to make sure the tag was signed.

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.

6 participants