prost-wkt icon indicating copy to clipboard operation
prost-wkt copied to clipboard

Upgrade prost crates to 0.13

Open nickpresta opened this issue 1 year ago • 2 comments

Upgrade to the latest prost version (0.13):

  • https://crates.io/crates/prost
  • https://crates.io/crates/prost-types
  • https://crates.io/crates/prost-build

Closes https://github.com/fdeantoni/prost-wkt/issues/67

nickpresta avatar Jul 10 '24 00:07 nickpresta

@fdeantoni Don't intend to pressure you - just wanted to let you know this PR exists and ask whether there's anything else I need to do as part of this change (e.g. CHANGELOG entry). Thanks!

nickpresta avatar Jul 11 '24 14:07 nickpresta

Many thanks for putting this PR together! I will review and if all looks good will try to create a new release in the next few days.

fdeantoni avatar Jul 11 '24 14:07 fdeantoni

no pressure

ytitov avatar Jul 15 '24 17:07 ytitov

@fdeantoni friendly ping? Would be nice to get this released!

djc avatar Jul 25 '24 14:07 djc

Hi all, apologies for the delay! I will make sure to push out an update this weekend.

fdeantoni avatar Jul 26 '24 08:07 fdeantoni

Should this be 0.6 release? Seems like updating the prost to 0.13 is breaking builds.

59   | #[derive(Clone, PartialEq, ::prost::Message)]
     |                            ^^^^^^^^^^^^^^^^ the trait `prost::Message` is not implemented for `prost_wkt_types::Duration`
     |
     = help: the following other types implement trait `prost::Message`:
               ()
               Acceptance
               Access
               AccountAccess
               ActivityExecutionResult
               ActivityFailureInfo
               ActivityHeartbeat
               ActivityPropertiesModifiedExternallyEventAttributes
             and 535 others
note: required by a bound in `prost::encoding::message::merge`
    --> /Users/h7kanna/.cargo/registry/src/index.crates.io-6f17d22bba15001f/prost-0.12.6/src/encoding.rs:1075:12

h7kanna avatar Jul 28 '24 20:07 h7kanna

Yup, this should definitely get a semver-incompatible release (and you probably want to yank 0.5.2).

djc avatar Jul 28 '24 21:07 djc

Were all the dependencies updated to Prost 0.13 and prost-wkt to 0.5.2? Although Prost has gone to 0.13, the implementation of prost-wkt has not changed significantly, hence I kept it at 0.5. I realize now that this may cause problems and it should go to 0.6. I will yank this release and make it 0.6.0 instead. Apologies for all the trouble caused!

fdeantoni avatar Jul 29 '24 05:07 fdeantoni

Hi All, I have yanked 0.5.2 and released it as 0.6.0 instead. I've added a CHANGELOG.md to document this as well. Please test it out and if any other problems encountered do let me know!!

fdeantoni avatar Jul 29 '24 06:07 fdeantoni

prost-wkt-build needs to be yanked as well

[[package]]
name = "prost-wkt-build"
version = "0.5.2"
source = "registry+https://github.com/rust-lang/crates.io-index"
checksum = "a06feedac9f90870ab1c46ac404cf3217ba42d220c62ff8f8119d4b1f685741c"
dependencies = [
 "heck 0.5.0",
 "prost 0.13.1",
 "prost-build 0.13.1",
 "prost-types 0.13.1",
 "quote",
]

h7kanna avatar Jul 29 '24 07:07 h7kanna

Apologies yes, prost-wkt-build and prost-wkt-types have been yanked now as well.

fdeantoni avatar Jul 29 '24 07:07 fdeantoni

All good now, Thank you.

h7kanna avatar Jul 29 '24 07:07 h7kanna