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.

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

Coloring non TTY terminal is broken since commit 86cb440 #679

Closed
ayalash opened this issue Sep 18, 2017 · 1 comment
Closed

Coloring non TTY terminal is broken since commit 86cb440 #679

ayalash opened this issue Sep 18, 2017 · 1 comment
Assignees
@ayalash
Milestone
1.5.0

Comments

@ayalash
Copy link
Collaborator

ayalash commented Sep 18, 2017

PR #667 added colorama.init() and colorama.deinit() to slash main_entry_point().
Apparently, this removes colors of non TTY terminals. The colors seems OK for TTY.
Checked with both arch and ubuntu.
@ayalash ayalash self-assigned this Sep 18, 2017
@ayalash
Copy link
Collaborator Author

ayalash commented Sep 18, 2017

@pierreluctg - FYI

ayalash added a commit that referenced this issue Sep 18, 2017
@ayalash
Fix coloring for non TTY stdout (fix #679)
e5e349e
@ayalash ayalash added this to the 1.5.0 milestone Sep 18, 2017
@vmalloc vmalloc mentioned this issue Jun 19, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ayalash ayalash

Labels
None yet
Projects
None yet
Milestone
1.5.0
Development

No branches or pull requests
1 participant
@ayalash