Skip to content

fix: correctly style rust logs with colors #29040

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

Merged
merged 1 commit into from
Apr 24, 2025
Merged

Conversation

lucacasonato
Copy link
Member

We upgraded env_logger, which uses a new color detection backend that is backwards incompatible with the old backend and does not detect GitHub Actions as a terminal that supports color. We now force env_logger to respect the color decision that deno_terminal has already made.

We upgraded `env_logger`, which uses a new color detection backend that is backwards incompatible with the old backend and does not detect GitHub Actions as a terminal that supports color. We now force env_logger to respect the color decision that `deno_terminal` has already made.
Copy link
Member

@dsherret dsherret left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dsherret dsherret merged commit 7b908e5 into main Apr 24, 2025
18 checks passed
@dsherret dsherret deleted the force_style_on_rust_logger branch April 24, 2025 19:06
dsherret pushed a commit that referenced this pull request Apr 24, 2025
We upgraded `env_logger`, which uses a new color detection backend that
is backwards incompatible with the old backend and does not detect
GitHub Actions as a terminal that supports color. We now force
env_logger to respect the color decision that `deno_terminal` has
already made.
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.

2 participants