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.

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

docs(vrl): add crc vrl function docs #21924

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

docs(vrl): add crc vrl function docs #21924

wants to merge 7 commits into from

Conversation

ivor11
Copy link

@ivor11 ivor11 commented Nov 30, 2024

Summary

add docs for crc32 vrl function
vectordotdev/vrl#1136

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.

Checklist

  • Please read our Vector contributor resources.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run dd-rust-license-tool write to regenerate the license inventory and commit the changes (if any). More details here.

References

vectordotdev/vrl#997

@ivor11 ivor11 requested review from a team as code owners November 30, 2024 20:32
@bits-bot
Copy link

bits-bot commented Nov 30, 2024 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions bot added the domain: external docs Anything related to Vector's external, public documentation label Nov 30, 2024
@ivor11 ivor11 mentioned this pull request Nov 30, 2024
Merged
11 tasks
@jszwedko jszwedko enabled auto-merge December 5, 2024 01:07
pront
pront requested changes Dec 5, 2024
View reviewed changes
Copy link
Member

@pront pront left a comment

Choose a reason for hiding this comment

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

Waiting for this improvement: vectordotdev/vrl#1136 (comment) (docs will have to be updated)

@ivor11 ivor11 changed the title feat(crc32): add crc32 vrl function docs feat(crc32): add crc vrl function docs Dec 10, 2024
auto-merge was automatically disabled December 10, 2024 22:29

Head branch was pushed to by a user without write access

@ivor11 ivor11 changed the title feat(crc32): add crc vrl function docs docs(vrl): add crc vrl function docs Dec 10, 2024
@ivor11 ivor11 requested a review from a team as a code owner December 10, 2024 22:48
ivor11 and others added 2 commits December 10, 2024 22:50
@ivor11
resolving conficts with master
4177f57
@ivor11
Update metadata
e0cffc3 
check-spelling run (pull_request_target) for feat/crc32

Signed-off-by: check-spelling-bot <[email protected]>
on-behalf-of: @check-spelling <[email protected]>
@ivor11 ivor11 requested a review from pront December 11, 2024 08:11
@pront pront added this to the Vector v0.44.0 milestone Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@estherk15 estherk15 estherk15 approved these changes

@jszwedko jszwedko jszwedko approved these changes

@pront pront Awaiting requested review from pront

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
domain: external docs Anything related to Vector's external, public documentation
Projects
None yet
Milestone
Vector v0.44.0
Development

Successfully merging this pull request may close these issues.
5 participants
@ivor11 @bits-bot @jszwedko @pront @estherk15