Skip to content

chore(deps): Disable default features for rust_decimal to avoid deps#1298

Merged
pront merged 1 commit intovectordotdev:mainfrom
JakubOnderka:rust-decimal-features
Feb 25, 2025
Merged

chore(deps): Disable default features for rust_decimal to avoid deps#1298
pront merged 1 commit intovectordotdev:mainfrom
JakubOnderka:rust-decimal-features

Conversation

@JakubOnderka
Copy link
Copy Markdown
Contributor

@JakubOnderka JakubOnderka commented Feb 25, 2025

Summary

This simple change will remove 21 unnecessary dependencies.

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?

Standard test case.

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.

Copy link
Copy Markdown
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.

Always satisfying to see deps removed 👏

@pront pront added the no-changelog Changes in this PR do not need user-facing explanations in the release changelog label Feb 25, 2025
@pront pront enabled auto-merge February 25, 2025 14:52
@pront pront added this pull request to the merge queue Feb 25, 2025
Merged via the queue into vectordotdev:main with commit 98c1463 Feb 25, 2025
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