Skip to content

V1.0 deprecations #498

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

Merged
merged 5 commits into from
Dec 7, 2018
Merged

V1.0 deprecations #498

merged 5 commits into from
Dec 7, 2018

Conversation

rnpridgeon
Copy link
Contributor

Note two remaining default.topic.config are already handled in #440

@rnpridgeon rnpridgeon requested a review from edenhill December 3, 2018 20:33
Copy link
Contributor

@edenhill edenhill left a comment

Choose a reason for hiding this comment

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

there's at least one default.topic.config that wasn't changed and without a comment on why

Copy link
Contributor

@edenhill edenhill left a comment

Choose a reason for hiding this comment

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

also fix the history, ff to master is not helpful.

print("Consumer error: {}".format(msg.error()))
continue

print('consumed: {0}'.format(msg.value()))
Copy link
Contributor

Choose a reason for hiding this comment

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

The positional {0} is not really needed, {} is fine and more consistent with other code.

Copy link
Contributor

@edenhill edenhill left a comment

Choose a reason for hiding this comment

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

LGTM!

@rnpridgeon rnpridgeon merged commit 590937f into master Dec 7, 2018
@rnpridgeon rnpridgeon deleted the v1.0-deprecations branch December 7, 2018 13:37
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