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

feat: Add support for forcing stdout ansi #305

Closed
wants to merge 1 commit into from
Closed

Conversation

Miuler
Copy link

@Miuler Miuler commented Mar 14, 2023

Add support for forcing stdout ansi. For example, the test in intellij not detect the stdout:

image

FIXED with force stdout

image

@Miuler
Copy link
Author

Miuler commented Mar 28, 2023

Resolve #307

@bconn98
Copy link
Collaborator

bconn98 commented Nov 30, 2023

I'm not sure it makes sense to make a new configuration value to do this when there are already long supported environment variables that do the same thing. @Miuler is there a reason you preferred a configuration based approach instead of using a Linux standard alternative?

@bconn98
Copy link
Collaborator

bconn98 commented Jan 26, 2024

@Miuler Please review the MR #335. If you're satisfied with the new capabilities, please close this PR.

@estk
Copy link
Owner

estk commented Feb 10, 2024

Closing since no response

@estk estk closed this Feb 10, 2024
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.

None yet

3 participants