Skip to content

Upgrade prost to 0.8.0 #121

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

Merged
merged 2 commits into from
Sep 28, 2021
Merged

Upgrade prost to 0.8.0 #121

merged 2 commits into from
Sep 28, 2021

Conversation

hug-dev
Copy link
Member

@hug-dev hug-dev commented Sep 27, 2021

This is to make our crates compile with the latest nightly Rust version.
Also performs a cargo update and regenerate the Protobuf files with
the latest version.

See parallaxsecond/parsec#514

This is to make our crates compile with the latest nightly Rust version.
Also performs a `cargo update` and regenerate the Protobuf files with
the latest version.

Signed-off-by: Hugues de Valon <[email protected]>
@ionut-arm
Copy link
Member

I'm guessing here that the minimum version of Rust we support in the service gets bumped to 1.53?

@hug-dev
Copy link
Member Author

hug-dev commented Sep 28, 2021

I'm guessing here that the minimum version of Rust we support in the service gets bumped to 1.53?

Yes! From the issue linked, it was confirmed that we can bump our MSRV in Parsec to 1.53.0 so starting from this crate I will update all the dependant ones and upgrade the PArsec CI system!

@hug-dev
Copy link
Member Author

hug-dev commented Sep 28, 2021

We will also be able to remove our cargo audit rules I believe?

Copy link
Member

@ionut-arm ionut-arm left a comment

Choose a reason for hiding this comment

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

Yes, let's do this!! I hope the changes in the generated files won't come back to bite us

Signed-off-by: Hugues de Valon <[email protected]>
@hug-dev hug-dev enabled auto-merge September 28, 2021 09:34
@hug-dev hug-dev merged commit e5b7f17 into parallaxsecond:main Sep 28, 2021
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.

3 participants