Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@soleinik-figment
Copy link
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

Copy link
Contributor

@maoueh maoueh left a comment

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
Copy link
Contributor

evaporei commented May 5, 2022

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

@soleinik-figment
Copy link
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