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

Core: Refactor log_error to unify color handling #100724

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Repiteo
Copy link
Contributor

@Repiteo Repiteo commented Dec 22, 2024

Takes many of the improvements proposed in the above PR, minus the somewhat contentious dim-color addition. Now uniformly detects if the environment is appropriate for colored output utilizing #91201's get_stdout_type(), and properly detects CI for colored output on GitHub Actions. The colors have also been unified so the standard versions (excluding gray) don't use the "intense" palette — this was already the case on Windows, but Mac/Linux mixed the two — as consoles automatically swap intensity based on if the text is bold or not

@Repiteo Repiteo added this to the 4.4 milestone Dec 22, 2024
@Repiteo Repiteo requested review from a team as code owners December 22, 2024 02:59
@Repiteo Repiteo force-pushed the core/log_error-refactor branch from 643187d to 163fca0 Compare December 22, 2024 14:03
• Can now detect & output color on CI
@Repiteo Repiteo force-pushed the core/log_error-refactor branch from 163fca0 to 5bd7cf1 Compare December 22, 2024 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant