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: fix CI builds #104

Merged
merged 7 commits into from
Aug 11, 2023
Merged

chore: fix CI builds #104

merged 7 commits into from
Aug 11, 2023

Conversation

sporkmonger
Copy link
Contributor

@sporkmonger sporkmonger commented Aug 10, 2023

CI builds seem to be failing due to the toolchain being set to an old version of Rust, out of date descriptors, and a number of accumulated clippy warnings.

@sporkmonger sporkmonger changed the title CI builds locked on old Rust version Fix CI builds Aug 10, 2023
@tustvold tustvold changed the title Fix CI builds chore: fix CI builds Aug 11, 2023
@tustvold tustvold merged commit 768c324 into influxdata:main Aug 11, 2023
7 of 9 checks passed
@sporkmonger sporkmonger deleted the fix-ci branch August 11, 2023 21:46
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.

None yet

2 participants