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(remap transform): add strip_ansi_escape_codes function #4170

Merged
merged 1 commit into from
Sep 29, 2020

Conversation

JeanMertz
Copy link
Contributor

.foo = strip_ansi_escape_codes(.foo)

I didn't add a behaviour test, since TOML doesn't support the \e escape character. I guess we could parse something similar to <null>, to make that work, but it didn't seem worth the investment for this one remap function (which is already tested in other ways).

closes #3754

@JeanMertz JeanMertz added type: feature A value-adding code addition that introduce new functionality. domain: vrl Anything related to the Vector Remap Language transform: remap Anything `remap` transform related labels Sep 28, 2020
@JeanMertz JeanMertz added this to the 2020-09-28 - Derezzed milestone Sep 28, 2020
@JeanMertz JeanMertz self-assigned this Sep 28, 2020
@JeanMertz JeanMertz merged commit 8c9a72b into master Sep 29, 2020
@JeanMertz JeanMertz deleted the jean/remap-strip_ansi_escape_codes branch September 29, 2020 11:06
mengesb pushed a commit to jacobbraaten/vector that referenced this pull request Dec 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain: vrl Anything related to the Vector Remap Language transform: remap Anything `remap` transform related type: feature A value-adding code addition that introduce new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New strip_ansi_escape_codes remap function
2 participants