-
-
Notifications
You must be signed in to change notification settings - Fork 265
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
deps: Update bit-vec
, downcast-rs
, ordered-float
, thiserror
#808
base: master
Are you sure you want to change the base?
Conversation
downcast-rs = "1.2" | ||
downcast-rs = "2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! To keep track I'm noting that parry upgraded downcast to version 2 in dimforge/parry#305 ; we'll want to update rapier to the future parry version to avoid duplicate dependencies
ordered-float = "4" | ||
thiserror = "1" | ||
ordered-float = "5" | ||
thiserror = "2" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! To keep track I'm noting that parry upgraded thiserror to version 2 in dimforge/parry#306 ; we'll want to update rapier to the future parry version to avoid duplicate dependencies
num-derive = "0.4" | ||
bitflags = "2" | ||
log = "0.4" | ||
ordered-float = "4" | ||
thiserror = "1" | ||
ordered-float = "5" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
parry didn't upgrade ordered-float yet so that should be blocking.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I submitted a PR for that last week: dimforge/parry#322
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Merged it. Keeping the discussion opened just to track dependencies.
No description provided.