Force rich to use color codes when pretty printing #1821
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #1819.
When piping,
rich
's default behavior is to suppress color codes. This is very useful except in the particular scenario that we want to use rich to wrap text with color codes, which happens to be the situation that we're using rich in since #1693. This PR forces rich to supply codes by treating it as a terminal.Unfortunately, I don't see how to test this automatically, but I was able to manually verify this with the following steps in WSL/Ubuntu from the jrnl source root:
My default journal has colors configured, so once I got this working, the colors appeared in the terminal after running the above commands.
Checklist
for the same issue.