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

feat: indicate wrapped lines #1

Merged
merged 17 commits into from
Feb 7, 2023
Merged

feat: indicate wrapped lines #1

merged 17 commits into from
Feb 7, 2023

Conversation

KyleKing
Copy link
Owner

@KyleKing KyleKing commented Feb 1, 2023

Attempting to indent each wrapped line for better readability. Can test with:

python -c 'import json; [print(json.dumps({f"{key}": "-" * 30 for key in range(10)})) for _ in range(5)]' | poetry run tail-jsonl

Opened discussion on rich: Textualize/rich#2792

Update: In the meantime, added a string prefix for readability

@KyleKing KyleKing self-assigned this Feb 1, 2023
@KyleKing KyleKing changed the title feat: minor updates feat: indent wrapped lines Feb 2, 2023
@KyleKing KyleKing marked this pull request as ready for review February 2, 2023 00:54
@KyleKing KyleKing marked this pull request as draft February 2, 2023 01:25
@KyleKing KyleKing changed the title feat: indent wrapped lines feat: indicate wrapped lines Feb 7, 2023
@KyleKing KyleKing marked this pull request as ready for review February 7, 2023 04:58
@KyleKing KyleKing merged commit dad146d into main Feb 7, 2023
@KyleKing KyleKing deleted the v1 branch February 7, 2023 12:33
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.

None yet

1 participant