Skip to content

bump(cargo)!: 🚀 upgrades available#1545

Merged
qmonnet merged 4 commits into
mainfrom
bump/cargo-upgrades
May 18, 2026
Merged

bump(cargo)!: 🚀 upgrades available#1545
qmonnet merged 4 commits into
mainfrom
bump/cargo-upgrades

Conversation

@hedgehog-dataplane-update
Copy link
Copy Markdown
Contributor

🚀 Upgrades available

name        old req compatible latest new req
====        ======= ========== ====== =======
dashmap     6.1.0   6.2.1      6.2.1  6.2.1  
prefix-trie 0.8.4   0.8.4      0.9.0  0.9.0  

⚠️ This Pull Request was automatically generated and should be carefully reviewed before acceptance. It may introduce breaking changes.

Signed-off-by: github-actions[bot] <224724778+hedgehog-dataplane-update[bot]@users.noreply.github.com>
Signed-off-by: github-actions[bot] <224724778+hedgehog-dataplane-update[bot]@users.noreply.github.com>
name    old req compatible latest new req
====    ======= ========== ====== =======
dashmap 6.1.0   6.2.1      6.2.1  6.2.1

Signed-off-by: github-actions[bot] <224724778+hedgehog-dataplane-update[bot]@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 18, 2026 04:48
@hedgehog-dataplane-update hedgehog-dataplane-update Bot requested a review from a team as a code owner May 18, 2026 04:48
@hedgehog-dataplane-update hedgehog-dataplane-update Bot added the dependencies Pull requests that update a dependency file label May 18, 2026
@hedgehog-dataplane-update hedgehog-dataplane-update Bot requested review from daniel-noland and removed request for a team May 18, 2026 04:48
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@qmonnet qmonnet force-pushed the bump/cargo-upgrades branch 2 times, most recently from 47255e1 to 1dd35d6 Compare May 18, 2026 11:45
@qmonnet
Copy link
Copy Markdown
Member

qmonnet commented May 18, 2026

The prefix-trie update is breaking, the tries no longer hold the prefixes (they're reconstructed), meaning we can no longer return pointers to it. I had Claude propagate the change.

name        old req compatible latest new req
====        ======= ========== ====== =======
prefix-trie 0.8.4   0.8.4      0.9.2  0.9.2

This change requires code adjustments. The migration notes from 0.8.*
to 0.9.0 include:

    * Prefixes are reconstructed from the tree location and no longer
      stored.
      Consequently, information in the host part of an address is no
      longer maintained.
      Additionally, the function yields prefixes by value instead of by
      reference.

This means we can no longer just return pointers to prefixes in the trie
when iterating over the entries, for example: we must reconstruct and
return owned versions of the prefixes.

[ Quentin: Bump from 0.9.0 to 0.9.2, have Claude fix the breakage ]

Link: https://github.com/tiborschneider/prefix-trie/releases/tag/v0.9.0
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Signed-off-by: github-actions[bot] <224724778+hedgehog-dataplane-update[bot]@users.noreply.github.com>
Signed-off-by: Quentin Monnet <qmo@qmon.net>
@qmonnet qmonnet force-pushed the bump/cargo-upgrades branch from 1dd35d6 to 8d3a449 Compare May 18, 2026 11:48
@qmonnet qmonnet added this pull request to the merge queue May 18, 2026
Merged via the queue into main with commit 5005ae5 May 18, 2026
35 checks passed
@qmonnet qmonnet deleted the bump/cargo-upgrades branch May 18, 2026 12:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants