Skip to content

chore: standardise dependencies #724

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Tony-Samuels
Copy link
Collaborator

No description provided.

proptest = { default-features = false, optional = true, features = ["std"], version = "1.0" }
rand-0_9 = { default-features = false, optional = true, package = "rand", version = "0.9" }
rocket = { default-features = false, optional = true, version = "0.5" }
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Rocket can't be renamed, as its derive requires the standard crate name.

Note: rocket doesn't seem to have been updated in over a year and despite that (looking at the crate graph), most users are still on rc versions. I think we should consider removing support for it in v2.

Cargo.toml Outdated
version-sync = { default-features = false, features = ["html_root_url_updated", "markdown_deps_updated"], version = "0.9" }

# Enable features for testing
rust_decimal = { path = ".", features = ["rand-0_9", "macros", "tokio-pg", "rocket-0_5-traits"] }
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Let me know if you don't like this, but it saves copying the same versions of everything again. Note that it doesn't work for crates like rand where we use additional features for testing.

@@ -210,7 +210,7 @@ fn decimal_to_string(b: &mut test::Bencher) {
})
}

#[cfg(feature = "postgres")]
#[cfg(feature = "db-postgres")]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

There is no postgres feature. I took a guess at what this was supposed to be, but haven't validated it.

rust_decimal_macros = { path = "macros", default-features = false, optional = true, version = "1" }
rkyv = { default-features = false, features = ["size_32", "std"], optional = true, version = "0.7.42" }
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

We have an example on how to integrate against rkyv 0.8. I don't think our v2 needs to support rkyv 0.7.

borsh = { default-features = false, features = ["derive", "unstable__schema"], optional = true, version = "1" }
bytes = { default-features = false, optional = true, version = "1" }
diesel = { default-features = false, optional = true, version = "2" }
ndarray-0_16 = { default-features = false, optional = true, package = "ndarray", version = "0.16" }
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Calling out the version bump from 0.15 to 0.16

@Tony-Samuels Tony-Samuels force-pushed the standardise-deps branch 2 times, most recently from 426987d to db5ee58 Compare April 21, 2025 19:42
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