Skip to content

Sergey/prost bump version to 0 10#3531

Merged
evaporei merged 6 commits into
graphprotocol:masterfrom
soleinik-figment:sergey/prost_bump_version_to_0_10
May 5, 2022
Merged

Sergey/prost bump version to 0 10#3531
evaporei merged 6 commits into
graphprotocol:masterfrom
soleinik-figment:sergey/prost_bump_version_to_0_10

Conversation

@soleinik-figment

Copy link
Copy Markdown
Contributor

Notes:

  1. tonic-build v0.7 does not support rustfmt on generated files, hence #[rustfmt::skip]
  2. tonic's Endpoint.connect_lazy() used to return Result and not anymore

@maoueh maoueh left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Seems good to me, I enable the CI to see that everything works.

@evaporei

evaporei commented May 5, 2022

Copy link
Copy Markdown
Contributor

Actually, can't you run cargo fmt after doing the build so the generated files are formatted?

@soleinik-figment

Copy link
Copy Markdown
Contributor Author

As an additional step before pushing to git hub? If yes - then yes... it's just anyone who will work with *.proto file will have to remember to do so :-) as generated files will not be formatted. If we really want - earlier tonic-build has this function so we could try to copy it into build.rs...

@evaporei
evaporei merged commit f4b4d55 into graphprotocol:master May 5, 2022
@soleinik-figment
soleinik-figment deleted the sergey/prost_bump_version_to_0_10 branch May 6, 2022 02:18
lutter added a commit that referenced this pull request May 6, 2022
Requirement was added in this PR:
#3531
lutter added a commit that referenced this pull request May 6, 2022
Requirement was added in this PR:
#3531
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