Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Remove ANSI escape sequences from output #162

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

ZexDC
Copy link

@ZexDC ZexDC commented May 19, 2023

Detect and remove ANSI escape sequences from output as mentioned here.

sruehl added a commit to sruehl/go-junit-report that referenced this pull request Jun 5, 2023
@sruehl
Copy link

sruehl commented Jun 5, 2023

@ZexDC this lib is also stripping \u009B. There is a discussion also on SO regarding those sequences. Might be good if really all sequences get stripped.

@ZexDC
Copy link
Author

ZexDC commented Jun 16, 2023

@ZexDC this lib is also stripping \u009B. There is a discussion also on SO regarding those sequences. Might be good if really all sequences get stripped.

Thanks, I added the changes to this PR.

@ZexDC
Copy link
Author

ZexDC commented Jun 20, 2023

@jstemmer please let me know if this can be merged :)

@jwx0925
Copy link

jwx0925 commented Feb 26, 2025

I encountered the same issue. And I test with the code from this PR, it worked fine. Could you please merge it and release a new version as soon as possible?
@jstemmer Thanks ^_^

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants