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

Logging: Added color support for compatible terminals #7181

Merged
merged 3 commits into from
Nov 25, 2020

Conversation

rafalou38
Copy link
Contributor

WT_SESSION => Windows terminal
COLORTERM == truecolor => VS Code terminal and others
PYCHARM_HOSTED => Pycharm Run tab
TERMINAL_EMULATOR == JetBrains-JediTerm => Pycharm terminal

"WT_SESSION" => Windows terminal
"COLORTERM == truecolor" => VS Code terminal and others
"PYCHARM_HOSTED" => Pycharm Run tab
"TERMINAL_EMULATOR == JetBrains-JediTerm" => Pycharm terminal
@welcome
Copy link

welcome bot commented Oct 28, 2020

Thanks for opening your first pull request here! 💖 Please check out our contributing guidelines.

@rafalou38
Copy link
Contributor Author

No it won't break it $term is still here but there is more supported terminals added

@tshirtman
Copy link
Member

Yeah, sorry i removed that comment, i had read too fast and missed the code after.

@tshirtman tshirtman merged commit 3d3e45d into kivy:master Nov 25, 2020
@welcome
Copy link

welcome bot commented Nov 25, 2020

Congrats on merging your first pull request! 🎉🎉🎉

@matham matham changed the title Added color support for compatible terminals Logging: Added color support for compatible terminals Nov 25, 2020
@matham matham added this to the 2.0.0 milestone Nov 25, 2020
@matham matham added the Component: core-app app, clock, config, inspector, logger, resources, modules, clock, base.py label Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: core-app app, clock, config, inspector, logger, resources, modules, clock, base.py
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants