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.

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

Amend Exchange.__repr__ to include class name #527

Merged
merged 1 commit into from Mar 12, 2023
Merged

Amend Exchange.__repr__ to include class name #527

merged 1 commit into from Mar 12, 2023

Conversation

zware
Copy link
Contributor

@zware zware commented Mar 10, 2023 •
edited

Also switch to f-strings rather than %-formatting, modelled after
Queue.__repr__.

@zware
Amend Exchange.__repr__ to include class name
a7296cc 
Also switch to f-strings rather than %-formatting, modelled after
Queue.__repr__.
@coveralls
Copy link

Coverage Status

Coverage: 88.862% (-0.05%) from 88.911% when pulling a7296cc on zware:exchange_repr_fixup into 83d5432 on mosquito:master.

@coveralls
Copy link

Coverage Status

Coverage: ?%. Remained the same when pulling a7296cc on zware:exchange_repr_fixup into 83d5432 on mosquito:master.

@mosquito mosquito self-requested a review March 10, 2023 22:35
@mosquito mosquito merged commit 12b6f29 into mosquito:master Mar 12, 2023
@zware zware deleted the exchange_repr_fixup branch March 13, 2023 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mosquito mosquito Awaiting requested review from mosquito

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@zware @coveralls @mosquito