Skip to content

fix(dev): add toml to codecs dev-dependencies#24766

Open
thomasqueirozb wants to merge 1 commit intomasterfrom
codecs-toml-dev-deps
Open

fix(dev): add toml to codecs dev-dependencies#24766
thomasqueirozb wants to merge 1 commit intomasterfrom
codecs-toml-dev-deps

Conversation

@thomasqueirozb
Copy link
Contributor

Summary

cargo test -p codecs fails to compile because tests in transformer.rs use the toml crate which was only available behind the syslog feature flag. Add toml as a workspace dev-dependency to fix this.

Vector configuration

NA

How did you test this PR?

cargo test -p codecs --no-run compiles successfully.

Change Type

  • Bug fix
  • New feature
  • Dependencies
  • 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

@thomasqueirozb thomasqueirozb requested a review from a team as a code owner February 25, 2026 16:17
@pront
Copy link
Member

pront commented Feb 25, 2026

Related: #24661 (comment)

@thomasqueirozb thomasqueirozb added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Feb 25, 2026
@thomasqueirozb thomasqueirozb added this pull request to the merge queue Feb 25, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 25, 2026
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.

2 participants