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

Neater logging with wrapping at 80 chars #13

Open
wants to merge 3 commits into
base: FEI-5987-2
Choose a base branch
from

Conversation

lillialexis
Copy link
Member

@lillialexis lillialexis commented Nov 15, 2024

Summary:

I wanted to indent the sub-messages so that they catch the eye better.

See screenshot
Screenshot 2024-11-15 at 2 31 06 PM

It's not perfect, since interpolated substrings could, in theory, still cause wrapping, but it's not important enough to try and find a method that successfully breaks along string at the word boundaries.

Issue: FEI-5987

Test plan:

Run ka-clone in window that is 80 wide and see it look neat

@lillialexis lillialexis self-assigned this Nov 15, 2024
@lillialexis lillialexis requested review from jeresig and a team November 15, 2024 22:32
Copy link
Member

@csilvers csilvers left a comment

Choose a reason for hiding this comment

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

So pretty!

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