Skip to content

Conversation

@HoneyryderChuck
Copy link
Collaborator

a few last minute signature improvements, and the version + changelog changes.

Testing Trusted Publishing as well through it.

@HoneyryderChuck HoneyryderChuck force-pushed the sig-updates branch 3 times, most recently from 00a6e6b to de232cd Compare April 3, 2025 22:15

module HTTP2
VERSION = "1.0.2"
VERSION = "1.1.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I think version gets bumped automatically with the publisher, doesn't it? I thought I read something about version.rb being assumed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

it doesn't (and I'm counting on it, as changes in this file trigger the release workflow).

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Copy link
Collaborator

Choose a reason for hiding this comment

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

https://bundler.io/guides/creating_gem.html#releasing-the-gem

This is what I was thinking of. I guess it's gem bump that could do this. So that can be in the CI if you wanted.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Not sure if that's the most feasible. You need a way to tell CI that you want to bump the version, and whether it's a patch/minor/major bump. Bumping locally and triggering the release flow on changes to version.rb seems more straightforward.

@HoneyryderChuck HoneyryderChuck merged commit 417f74b into main Apr 10, 2025
10 checks passed
@HoneyryderChuck HoneyryderChuck deleted the sig-updates branch April 10, 2025 07:41
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.

3 participants