Skip to content

docs(external): vrl encode/decode_lz4 documentation update sibling PR #23378

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

docs(external): vrl encode/decode_lz4 documentation update sibling PR #23378

wants to merge 9 commits into from

Conversation

jlambatl
Copy link
Contributor

Summary

This pull request is a sibling PR for vectordotdev/vrl#1447, which updates and changes some of the encode_lz4 and decode_lz4 VRL functions.

Vector configuration

Documentation update

How did you test this PR?

Documentation only, I have run the documentation checks in the scripts folder.

Change Type

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

Is this a breaking change?

  • Yes
  • No

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.

References

@jlambatl jlambatl requested a review from a team as a code owner July 15, 2025 10:01
@github-actions github-actions bot added the domain: external docs Anything related to Vector's external, public documentation label Jul 15, 2025
@jlambatl jlambatl changed the title enhancement(lz4 encode/decode): Documentation update sibling PR docs(external): vrl encode/decode_lz4 documentation update sibling PR Jul 15, 2025
@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Jul 15, 2025
@pront pront enabled auto-merge July 15, 2025 13:59
@pront pront disabled auto-merge July 15, 2025 13:59
@pront pront self-assigned this Jul 15, 2025
@buraizu buraizu self-assigned this Jul 15, 2025
buraizu
buraizu approved these changes Jul 15, 2025
View reviewed changes
Copy link
Contributor

@buraizu buraizu left a comment

Choose a reason for hiding this comment

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

Approving with a couple of minor suggestions

@buraizu buraizu removed their assignment Jul 15, 2025
jlambatl and others added 4 commits July 16, 2025 08:27
@jlambatl @buraizu
Update website/cue/reference/remap/functions/decode_lz4.cue
36fac2a 
much better use of grammar

Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>
@jlambatl @buraizu
Update website/cue/reference/remap/functions/encode_lz4.cue
d110dfe 
much better use of grammar

Co-authored-by: Bryce Eadie <bryce.eadie@datadoghq.com>
@pront
Merge branch 'master' into update_lz4_documentation
5470fbc
@pront
cargo update -p vrl
79208e8
@pront pront requested a review from a team as a code owner July 17, 2025 15:04
@pront pront enabled auto-merge July 17, 2025 15:04
thomasqueirozb
thomasqueirozb reviewed Jul 18, 2025
View reviewed changes
thomasqueirozb
thomasqueirozb reviewed Jul 18, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@thomasqueirozb thomasqueirozb thomasqueirozb left review comments

@buraizu buraizu buraizu approved these changes

At least 1 approving review is required to merge this pull request.

Assignees

@pront pront

Labels
domain: external docs Anything related to Vector's external, public documentation no-changelog Changes in this PR do not need user-facing explanations in the release changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@jlambatl @buraizu @thomasqueirozb @pront