Skip to content

chore(deps): bump serde_with to 3.21.0#10838

Merged
tmu0 merged 2 commits into
masterfrom
@tmu0/bump-serde-with
Jul 21, 2026
Merged

chore(deps): bump serde_with to 3.21.0#10838
tmu0 merged 2 commits into
masterfrom
@tmu0/bump-serde-with

Conversation

@tmu0

@tmu0 tmu0 commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

What

Bumps serde_with from 1.14.0 to 3.21.0 (workspace Cargo.toml + bazel/rust.MODULE.bazel crate spec) and repins the lock files.

Details

  • The workspace previously pinned serde_with = "1.14.0" while a transitive 3.x was pulled in via ic-bn-lib, so the tree carried two major versions. This bump collapses both onto a single 3.21.0 and drops the now-unused 1.x subtree (darling 0.13, strsim 0.10, serde_with_macros 1.5) — hence the large number of deletions in the lock files.
  • No source changes are required: the APIs in use (serde_as, DisplayFromStr, rust::double_option, serde_with::Bytes) are unchanged across the major bump.

Verification

  • cargo check passes for the direct users (config_types, ic-types, rosetta-core).
  • cargo clippy -D warnings clean on config_types and rosetta-core.
  • Bazel repin (CARGO_BAZEL_REPIN) regenerated Cargo.Bazel.{toml,json}.lock; @crate_index//:serde_with (3.21.0) builds.

(config_tool and full Bazel rdeps build/test were not run locally — this host is macOS without the internal build infra, and config_tool has a pre-existing Linux-only import gate. Left to CI.)

🤖 Generated with Claude Code

Update serde_with from 1.14.0 to 3.21.0 in the workspace Cargo.toml and
the Bazel crate spec, and repin the Cargo/Bazel lock files. This also
collapses the previously duplicated transitive serde_with 3.x (pulled in
via ic-bn-lib) onto a single 3.21.0, dropping the now-unused 1.x subtree
(darling 0.13, strsim 0.10, serde_with_macros 1.5).

The APIs in use (serde_as, DisplayFromStr, rust::double_option and
serde_with::Bytes) are unchanged across the major bump, so no source
changes are required.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the workspace’s serde_with dependency to a single, consistent 3.x version across Cargo and Bazel, removing the previously duplicated 1.x subtree from the resolved dependency graph and repinning the relevant lock files.

Changes:

  • Bump workspace serde_with from 1.14.0 to 3.21.0 in Cargo.toml.
  • Update Bazel Rust crate specification for serde_with to ^3.21.0.
  • Repin Cargo and Cargo-Bazel lock files, dropping now-unused serde_with 1.x-related dependencies and updating associated transitive versions.

Reviewed changes

Copilot reviewed 2 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
Cargo.toml Updates the workspace dependency pin for serde_with to 3.21.0.
Cargo.lock Collapses to a single serde_with 3.21.0 entry and removes the obsolete 1.x subtree and related transitive deps.
Cargo.Bazel.toml.lock Repins Bazel/Cargo lock data to match the new resolved dependency set (including serde_with 3.21.0).
Cargo.Bazel.json.lock Repins Bazel/Cargo JSON lock data and updates resolved crate IDs/checksums for the new graph.
bazel/rust.MODULE.bazel Updates the Bazel crate spec for serde_with to ^3.21.0.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Resolve lock-file conflicts from the hpke 0.13 migration (#10833) by
regenerating Cargo.lock and Cargo.Bazel.{toml,json}.lock on top of the
current master baseline, keeping serde_with unified at 3.21.0.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@tmu0
tmu0 marked this pull request as ready for review July 20, 2026 15:34
@tmu0
tmu0 requested a review from a team as a code owner July 20, 2026 15:34
@tmu0
tmu0 enabled auto-merge July 20, 2026 15:34
@github-actions github-actions Bot added the @idx label Jul 20, 2026
@tmu0
tmu0 added this pull request to the merge queue Jul 21, 2026
Merged via the queue into master with commit 59716f4 Jul 21, 2026
40 checks passed
@tmu0
tmu0 deleted the @tmu0/bump-serde-with branch July 21, 2026 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants