-
Notifications
You must be signed in to change notification settings - Fork 330
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
Bump to 2.6.0 #311
Bump to 2.6.0 #311
Conversation
@gabrieljoelc the change log looks OK. A bit more detail (à la Bunny) would be better but we can worry about that after the release. |
Pushed to rubygems.org: https://rubygems.org/gems/sneakers/versions/2.6.0. |
@gabrieljoelc just as we expected, #279 introduced a subtle change that can break things. I fixed one of them in #312 (too early to tell how many will run into it) and created a This raises a question: in Bunny, older series branches are merged into master. Is it something you'd subscribe to? Some are big believers in cherry-picking from master but in many years maintaining Bunny I tend to first patch the oldest branch. This way you cannot forget to backport. |
@michaelklishin yes, that looks like the right hotfix. I'm not sure I'm following you. When I created the 2.6.0 release branch, I had the feeling that I wished we were just doing git flow:
|
@michaelklishin do you want me to create a hotfix release with 7e96d48 today? |
@gabrieljoelc in Bunny there are stable release branches (e.g. Bunny does the former. RabbitMQ currently does the former but there's an ongoing debate on our team about switching to the latter at some point. Both can work but the former makes it impossible to forget to propagate a change to all branches. |
@gabrieljoelc sure, feel free to release |
See the diff between v2.5.0 and master. Looks like a minor version bump to me. Proposed changelog/release notes:
Sneaker.spawn
exits withexit_code
1
when no configuration is provided Exit the spawner with a status code. #280thread
gem withconcurrent-ruby
gem Replace gem thread with concurrent-ruby #279Is #279 the one that has the greatest chance of side-effects in production?