Skip to content

chore: upgrade all dependencies#3354

Draft
zvolin wants to merge 1 commit intozvolin/bump-rust-toolchainfrom
zvolin/upgrade-deps
Draft

chore: upgrade all dependencies#3354
zvolin wants to merge 1 commit intozvolin/bump-rust-toolchainfrom
zvolin/upgrade-deps

Conversation

@zvolin
Copy link
Copy Markdown
Contributor

@zvolin zvolin commented Apr 20, 2026

We cannot upgrade rand to 0.10 because primitive-types lock us at 0.8.5. This also blocks fake upgrade.

use super::*;

pub async fn bind_ephemeral() -> (SocketAddr, TcpListener) {
let listener = TcpListener::bind(("127.0.0.1", 0))
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

that's literally what warp::Server::bind does now, create new tokio::TcpListener and calls incoming on the Server. All the other bind_xyz methods were removed

"expected a list of name-value pairs",
));
};
meta_list.parse_nested_meta(|meta| {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

https://github.com/dtolnay/syn/releases/tag/2.0.0

a new parsing library called syn::meta, and the parse_nested_meta method on Attribute

Comment thread crates/storage/Cargo.toml
rand = { workspace = true }
rayon = { workspace = true }
rusqlite = { workspace = true, features = ["bundled", "array", "hooks"] }
rusqlite = { workspace = true, features = ["bundled", "array", "hooks", "fallible_uint"] }
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

rusqlite/rusqlite#1722

sqlite can only store numbers up to i64::MAX, maybe we should consider checking if we shouldn't migrate the code to ensure that is always the case? This feature flag gives preserves current logic tho

Comment thread crates/version/build.rs
//! Pathfinder build script.
//!
//! Just sets up `vergen` to query our git information for the build.
//! Just sets up `vergen_gitcl` to query our git information for the build.
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@zvolin zvolin marked this pull request as ready for review April 20, 2026 13:31
@zvolin zvolin requested a review from a team as a code owner April 20, 2026 13:31
@zvolin zvolin marked this pull request as draft April 20, 2026 13:32
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.

1 participant