Skip to content

Conversation

@pront
Copy link
Member

@pront pront commented Oct 1, 2025

Summary

This could be exposed to the VRL function as well, but I have no need for this and no one requested it so far.

Change Type

  • Bug fix
  • New feature
  • Non-functional (chore, refactoring, docs)
  • Performance

Is this a breaking change?

  • Yes
  • No

How did you test this PR?

Added test.

Does this PR include user facing changes?

  • Yes. Please add a changelog fragment based on
    our guidelines.
  • No. A maintainer will apply the "no-changelog" label to this PR.

Checklist

  • Our CONTRIBUTING.md is a good starting place.
  • If this PR introduces changes to LICENSE-3rdparty.csv, please
    run dd-rust-license-tool write and commit the changes. More details here.
  • For new VRL functions, please also create a sibling PR in Vector to document the new function.

References

@pront pront requested a review from a team as a code owner October 1, 2025 14:21
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Oct 1, 2025
@pront pront enabled auto-merge October 1, 2025 14:44
@pront pront added this pull request to the merge queue Oct 1, 2025
Merged via the queue into main with commit 1559ff0 Oct 1, 2025
13 of 15 checks passed
@pront pront deleted the pront-proto-encode-json-option branch October 1, 2025 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-changelog Changes in this PR do not need user-facing explanations in the release changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants