Skip to content

Bump toml_edit to 0.25 - #155

Open
glamberson wants to merge 1 commit into
SergioBenitez:masterfrom
lamco-admin:bump-toml-edit-0.25
Open

Bump toml_edit to 0.25#155
glamberson wants to merge 1 commit into
SergioBenitez:masterfrom
lamco-admin:bump-toml-edit-0.25

Conversation

@glamberson

Copy link
Copy Markdown

Bumps toml_edit from 0.22 (two years old) to the latest stable 0.25.11.

The parse and serde features used here, and the single call site toml_edit::de::from_str in src/providers/data.rs, are stable across 0.22 through 0.25. One-line Cargo.toml change, no source edits required.

Verification

  • cargo build --all-features: clean
  • cargo test --all-features: 143 passed, 0 failed

cargo clippy --all-features --all-targets -- -D warnings shows 48 pre-existing result_large_err errors on master. Those are unrelated to this change and reproducible on vanilla master.

Relation to PR #145

This PR supersedes #145 (by @Krysztal112233, open since 2025-07-22), which proposed the same bump to 0.23. Going straight to 0.25 avoids a second bump PR later. #145 can be closed in favour of this one.

Context

The published 0.10.19 crate still depends on toml = "0.8", which blocks downstream migrations to toml 0.9 (notably the current Debian Rust team transition tracked at debcargo-conf issue #147). Master has already switched to toml_edit, but the unreleased state means the transition is effectively stuck behind a future release. Merging this bump and cutting a release resolves the downstream friction cleanly.

toml_edit 0.22 is two years old. The latest stable is 0.25.11. The
bump is a one-line Cargo.toml change: the `parse` and `serde` features
Figment uses and the `toml_edit::de::from_str` call in
src/providers/data.rs are stable across 0.22 -> 0.25, so no source
changes are required.

PR SergioBenitez#145 (open since 2025-07-22) proposed a bump to 0.23. This PR
supersedes that one by going to the latest stable 0.25 instead.

Verified:
  cargo build --all-features
  cargo test --all-features            (143 passed, 0 failed)

Note: cargo clippy --all-features --all-targets -- -D warnings shows
48 pre-existing `result_large_err` errors on master that are unrelated
to this change. The bump itself introduces no new clippy findings.
@lmmx

lmmx commented Apr 20, 2026

Copy link
Copy Markdown

@glamberson

glamberson commented Apr 20, 2026

Copy link
Copy Markdown
Author

@lmmx thanks for mirroring it to figment2 and cutting v0.11.5. Appreciated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants