[repo] Bump rust version and dependencies#2532
Conversation
ekiwi
left a comment
There was a problem hiding this comment.
I scanned through it, and it looks good. Basically mostly a number of format string changes.
|
Incidentally there is still a future incompatibility warning because |
sampsyo
left a comment
There was a problem hiding this comment.
Thanks for doing this, @EclecticGriffin! The string-templating changes, while sensible, are a bit of a slog. I'll go ahead and merge this now!
About the morty: unfortunately, this is such a useful dependency that it would be a shame to just try to remove it... maybe we should look to see if it is feasible to contribute a version bump?
|
I wasted a little bit of time looking into this and ended up opening pulp-platform/morty#65. However, along the way, I noticed that the git trunk for |
|
Oh cool! Glad it was easy to resolve. Was worried we'd have to make a fork and bump a ton of dependencies ourself |
I've bumped the rust version to
1.88and also updated the versions of our rust dependencies. As this updatesclippy, that means a bunch of new lints are active now, so I had to go and correct those as well. The vast majority are format string related.Unfortunately that makes for a bit of a big PR.