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

Bump to version 0.13.0 #470

Merged
merged 2 commits into from
Jun 20, 2018
Merged

Bump to version 0.13.0 #470

merged 2 commits into from
Jun 20, 2018

Conversation

pawelchcki
Copy link
Contributor

No description provided.

@pawelchcki pawelchcki changed the base branch from master to 0.13-dev June 20, 2018 15:05
@pawelchcki pawelchcki requested a review from delner June 20, 2018 15:05
Copy link
Contributor

@delner delner left a comment

Choose a reason for hiding this comment

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

Need to make a few amendments to the changelog.

CHANGELOG.md Outdated
- Request and response header tags to Rack (#389)
- MySQL2 integration (#453) (@jamiehodge)
- Sidekiq tag job delay (#458) (@gottfrois)
- Sinatra tag request id (#427)
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think we actually added a request_id tag at all. We added "Request and response headers to Sinatra". This also needs to list #375.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We do tag request id, but its because its listed as one of the default headers we tag. Yeah but you're right the description needs to be expanded

Copy link
Contributor

Choose a reason for hiding this comment

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

Well what we did for Sinatra is consistent with what we did for Rack. So the comment should probably look the same.

CHANGELOG.md Outdated
- ActiveSupport::Notifications::Event helper for event tracing (#400)
- Request and response header tags to Rack (#389)
- MySQL2 integration (#453) (@jamiehodge)
- Sidekiq tag job delay (#458) (@gottfrois)
Copy link
Contributor

Choose a reason for hiding this comment

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

This has the wrong PR #458 should be #443. This also needs to list #418.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

😱 Thanks for spotting that !

CHANGELOG.md Outdated

### Fixed

- Excon close spans on errors (#426) (@jeffjo)
Copy link
Contributor

Choose a reason for hiding this comment

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

We usually do not list fixes for things that are being released for the first time. Let's remove this line, merge both the PR and the author into "Excon integration" above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

👍

@pawelchcki pawelchcki force-pushed the bump_to_version_0.13.0 branch 2 times, most recently from e37fc3c to 7d3915d Compare June 20, 2018 15:43
CHANGELOG.md Outdated
- Sequel integration (supporting Ruby 2.0+) (#171, #367) (@randy-girard, @twe4ked, @palin)
- gRPC integration (supporting Ruby 2.2+) (#379, #403) (@Jared-Prime)
- ActiveModelSerializers integration (#340) (@sullimander)
- Excon integration (#211, #426) (@walterking,@jeffjo)
Copy link
Contributor

Choose a reason for hiding this comment

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

Need to add a space between the names.

@pawelchcki pawelchcki merged commit ede0772 into 0.13-dev Jun 20, 2018
@pawelchcki pawelchcki deleted the bump_to_version_0.13.0 branch June 20, 2018 17:31
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