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

Fix: Apply f-string binlogstream.py L135:L141 #570

Merged

dongwook-chan merged 1 commit into julien-duponchelle:main from HongGeonUi:enhancement/f-string/HongGeonUi3

Oct 21, 2023

Contributor

HongGeonUi commented Oct 20, 2023

I found string format in binlogstream.py.
So I convert to f-string that lines.


          Fix: Apply f-string binlogstream.py L135:L141

8f5ac8a

dongwook-chan approved these changes

View reviewed changes

Collaborator

dongwook-chan left a comment

Great catch! Thanks for submitting the PR. You've made the code more readable and modern. Please don't hesitate to create a PR and ask any questions.

dongwook-chan merged commit fbeaead into julien-duponchelle:main

6 checks passed

dongwook-chan added the hacktoberfest-accepted label

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hacktoberfest-accepted