Skip to content

Pubsub: remove debugging print statements. #7842

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.

Sign up for GitHub

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

Jump to bottom
Closed
tseaver wants to merge 1 commit into from
Closed

Pubsub: remove debugging print statements. #7842

tseaver wants to merge 1 commit into from

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented May 1, 2019

Closes #7841.

Please review ASAP to unbreak CI.

@tseaver tseaver added api: pubsub Issues related to the Pub/Sub API. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release. labels May 1, 2019
@tseaver tseaver requested review from crwilcox and tswast May 1, 2019 22:48
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 1, 2019
plamut
plamut approved these changes May 1, 2019
View reviewed changes
Copy link
Contributor

@plamut plamut left a comment

Choose a reason for hiding this comment

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

Beat me to it. LGTM!

@tseaver
Copy link
Contributor Author

tseaver commented May 1, 2019

Superseded by #7843.

@tseaver tseaver closed this May 1, 2019
@tseaver tseaver deleted the 7841-pubsub-remove-debug-prints branch May 2, 2019 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tswast tswast Awaiting requested review from tswast

@crwilcox crwilcox Awaiting requested review from crwilcox

@sduskis sduskis Awaiting requested review from sduskis

@anguillanneuf anguillanneuf Awaiting requested review from anguillanneuf

1 more reviewer

@plamut plamut plamut approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement. priority: p1 Important issue which blocks shipping the next release. Will be fixed prior to next release.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Pub/Sub: System test failures on Python 2.7 (invalid syntax)

3 participants

@tseaver @plamut @googlebot