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

Fix two links in CHANGELOG #296

Merged
merged 1 commit into from
Oct 7, 2022

Conversation

coopergillan
Copy link
Contributor

Description

Update the text of two links to match the actual link.

Update the text of two links to match the actual link.
@@ -3,13 +3,13 @@
## Unreleased

## v2.8.2
* Handles ErroneousStateError, in previous versions the consumer would do several unecessary group leave/joins. The log level is also changed to WARN instead of ERROR. ([#285](https://github.com/zendesk/racecar/pull/295))
* Handles ErroneousStateError, in previous versions the consumer would do several unecessary group leave/joins. The log level is also changed to WARN instead of ERROR. ([#295](https://github.com/zendesk/racecar/pull/295))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Looking at both #285 and #295, it looks like this should definitely be #295.


## v2.8.1
* Adds new ErroneousStateError to racecar in order to give more information on this new possible exception.

## v2.8.0
* Update librdkafka version from 1.8.2 to 1.9.0 by upgrading from rdkafka 0.10.0 to 0.12.0. ([#283](https://github.com/zendesk/racecar/pull/293))
* Update librdkafka version from 1.8.2 to 1.9.0 by upgrading from rdkafka 0.10.0 to 0.12.0. ([#293](https://github.com/zendesk/racecar/pull/293))
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same here: looking at the description, this should definitely be #293.

@coopergillan
Copy link
Contributor Author

@leonmaia - does this change look good to you? Looks like the builds won't run unless someone kicks them off.

@coopergillan
Copy link
Contributor Author

@darena-zendesk - are you able to kick off these specs?

Copy link
Collaborator

@deepredsky deepredsky left a comment

Choose a reason for hiding this comment

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

Thanks for fixing the links 👍

@deepredsky
Copy link
Collaborator

@coopergillan if you rebase/recommit this should trigger the CI

jbdietrich
jbdietrich previously approved these changes Oct 7, 2022
@jbdietrich jbdietrich dismissed their stale review October 7, 2022 09:17

Trying to get CI to run

@deepredsky deepredsky closed this Oct 7, 2022
@deepredsky deepredsky reopened this Oct 7, 2022
@jbdietrich jbdietrich merged commit cea1a63 into zendesk:master Oct 7, 2022
@coopergillan coopergillan deleted the fix-changelog-link-typo branch April 26, 2023 21:18
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