Skip to content
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

chore: update prost to 0.12 #111

Merged
merged 3 commits into from
Sep 17, 2023
Merged

Conversation

jeffreyssmith2nd
Copy link
Contributor

@jeffreyssmith2nd jeffreyssmith2nd commented Sep 9, 2023

Closes #110, this change makes pbjson compatible with the latest prost (0.12)

@brocaar
Copy link

brocaar commented Sep 12, 2023

Thanks for looking into this @jeffreyssmith2nd 👍

Copy link
Contributor

@tustvold tustvold left a comment

Choose a reason for hiding this comment

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

Sorry for the delay in getting to this

@tustvold tustvold merged commit 0854380 into influxdata:main Sep 17, 2023
Farhad-Shabani added a commit to informalsystems/pbjson that referenced this pull request Dec 20, 2023
* chore: fix CI builds (influxdata#104)

* chore: update prost to 0.12 (influxdata#111)

* chore: update prost to 0.12

* chore: fix lint

* chore: update descriptors

---------

Co-authored-by: Raphael Taylor-Davies <[email protected]>

* fix: parsing numbers more than i32 and u32 (influxdata#87)

* fix: parsing numbers more than i32 and u32

* fix: f64::MANTISSA_DIGITS instead of 53

---------

Co-authored-by: Raphael Taylor-Davies <[email protected]>

* fix: Resolve Envoy xDS proto issues (influxdata#103)

* fix: Escape Self since it is a reserved keyword

* fix: Allow duplicate numbers in enums when aliases are enabled

* fix: Avoid field name collisions with the map ident

* fix: follow cargo best practice in build.rs script (influxdata#90)

Using env! inside the build.rs script makes it difficult to build this
package with other build tools like bazel which sandbox execution, since
the source directory might be at different paths between when the
build.rs file is compiled, and when the resulting build script is
executed.

Cargo recommends that CARGO_MANIFEST_DIR is read at runtime instead:
https://doc.rust-lang.org/cargo/reference/environment-variables.html#environment-variables-cargo-sets-for-build-scripts

The above links also includes the promise that Cargo will set the cwd to
the crate's directory. This patch uses this to find the descriptors.bin
file.

Co-authored-by: Raphael Taylor-Davies <[email protected]>

* chore(deps): update itertools requirement from 0.10 to 0.11 (influxdata#98)

Updates the requirements on [itertools](https://github.com/rust-itertools/itertools) to permit the latest version.
- [Changelog](https://github.com/rust-itertools/itertools/blob/master/CHANGELOG.md)
- [Commits](rust-itertools/itertools@v0.10.0...v0.11.0)

---
updated-dependencies:
- dependency-name: itertools
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Raphael Taylor-Davies <[email protected]>

* chore(deps): update base64 requirement from 0.13 to 0.21 (influxdata#89)

* chore(deps): update base64 requirement from 0.13 to 0.21

Updates the requirements on [base64](https://github.com/marshallpierce/rust-base64) to permit the latest version.
- [Release notes](https://github.com/marshallpierce/rust-base64/releases)
- [Changelog](https://github.com/marshallpierce/rust-base64/blob/master/RELEASE-NOTES.md)
- [Commits](marshallpierce/rust-base64@v0.13.0...v0.21.0)

---
updated-dependencies:
- dependency-name: base64
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>

* fix: fix removed methods

* fix: indifferent padding

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Raphael Taylor-Davies <[email protected]>

* chore: prepare 0.6.0 (influxdata#112)

* fix: prost deprecations (influxdata#113)

* chore: patch release pjson-build 0.6.2 (influxdata#114)

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Bob Aman <[email protected]>
Co-authored-by: Jeffrey Smith II <[email protected]>
Co-authored-by: Raphael Taylor-Davies <[email protected]>
Co-authored-by: Nikolay Alexandrov <[email protected]>
Co-authored-by: Thomas Gibson-Robinson <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Raphael Taylor-Davies <[email protected]>
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.

Compatibility issue with prost 0.12?
3 participants