Skip to content

Conversation

cpprhtn
Copy link
Contributor

@cpprhtn cpprhtn commented Aug 30, 2025

Summary

PascalCase and CamelCase are mixed.

For consistency, PascalCase has been adopted for all Titles.

Vector configuration

How did you test this PR?

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

Related: #23640

Notes

  • Please read our Vector contributor resources.
  • Do not hesitate to use @vectordotdev/vector to reach out to us regarding this PR.
  • Some CI checks run only after we manually approve them.
    • We recommend adding a pre-push hook, please see this template.
    • Alternatively, we recommend running the following locally before pushing to the remote branch:
      • cargo fmt --all
      • cargo clippy --workspace --all-targets -- -D warnings
      • cargo nextest run --workspace (alternatively, you can run cargo test --all)
  • After a review is requested, please avoid force pushes to help us review incrementally.
    • Feel free to push as many commits as you want. They will be squashed into one before merging.
    • For example, you can run git merge origin master and git push.
  • If this PR introduces changes Vector dependencies (modifies Cargo.lock), please
    run cargo vdev build licenses to regenerate the license inventory and commit the changes (if any). More details here.

jszwedko and others added 2 commits September 17, 2021 09:25
Signed-off-by: Jesse Szwedko <[email protected]>
@cpprhtn cpprhtn requested review from a team as code owners August 30, 2025 09:25
@pront
Copy link
Member

pront commented Sep 2, 2025

There are merge conflicts with origin/master

@thomasqueirozb thomasqueirozb added the meta: awaiting author Pull requests that are awaiting their author. label Sep 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
meta: awaiting author Pull requests that are awaiting their author.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants