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

Make the prints the same for all brokers #331

Open
wants to merge 15 commits into
base: main
Choose a base branch
from
Open

Conversation

PZES
Copy link
Contributor

@PZES PZES commented Aug 21, 2024

WIP

@PZES PZES marked this pull request as draft August 21, 2024 22:41
@PZES PZES marked this pull request as ready for review August 21, 2024 23:31
@PZES PZES closed this Aug 22, 2024
@NelsonDane
Copy link
Owner

Why was this closed? @PZES

@PZES
Copy link
Contributor Author

PZES commented Aug 22, 2024

i thoght i rechanged to a draft did not mean to close it

@PZES PZES reopened this Aug 22, 2024
@PZES PZES marked this pull request as draft August 22, 2024 17:27
@PZES
Copy link
Contributor Author

PZES commented Aug 28, 2024

Formats all the prints for all brokers to be the same and prints a summary at the end

@PZES PZES marked this pull request as ready for review August 28, 2024 20:38
@ImNotOssy
Copy link
Contributor

@PZES i dont think it works properly
image

@PZES
Copy link
Contributor Author

PZES commented Sep 6, 2024

@ImNotOssy it does not work for dry also publicAPI does not work for dry mode

@matthew55 matthew55 added enhancement New feature or request low-priority Probably won't be worked on anytime soon, but hopefully one day. labels Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request low-priority Probably won't be worked on anytime soon, but hopefully one day.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants