Skip to content

Conversation

@tottoto
Copy link
Contributor

@tottoto tottoto commented Nov 1, 2025

Makes chrono optional dependency.

@tottoto tottoto requested a review from a team as a code owner November 1, 2025 01:21
@tottoto tottoto force-pushed the make-chrono-optional-dependency branch from f3d75ac to a9f655a Compare November 2, 2025 07:44
Cargo.toml Outdated
anyhow = { version = "1.0", features = ["backtrace"] }
approx = "0.5.1"
backtrace = { version = "0.3.68" }
chrono = { version = "0.4.32", default-features = false }
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
chrono = { version = "0.4.32", default-features = false }
chrono = { version = "0.4.32", default-features = false, features = ["std"] }

One of our tests needs this feature enabled

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed.

@tottoto tottoto force-pushed the make-chrono-optional-dependency branch from a9f655a to dfdff87 Compare November 4, 2025 03:35
Copy link
Contributor

@isabelatkinson isabelatkinson left a comment

Choose a reason for hiding this comment

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

lgtm!

@isabelatkinson isabelatkinson merged commit 1a0607b into mongodb:main Nov 4, 2025
18 of 20 checks passed
@tottoto tottoto deleted the make-chrono-optional-dependency branch November 4, 2025 21:52
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