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

Skip to content

feat(chain): Arweave integration #3465

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 21 commits into from
May 11, 2022
Merged

feat(chain): Arweave integration #3465

merged 21 commits into from
May 11, 2022

Conversation

@clearloop
Copy link
Contributor

@clearloop clearloop commented Apr 13, 2022 •
edited
Loading

Changes

  • Add Arweave types
  • Add Arweave runtime abi
  • Implement BlockChain trait for Arweave
  • .

Tests

clearloop added 15 commits April 14, 2022 04:21
@clearloop clearloop marked this pull request as ready for review April 26, 2022 10:07
maoueh
maoueh previously requested changes Apr 26, 2022
View reviewed changes
@clearloop
Copy link
Contributor Author

clearloop commented Apr 28, 2022

Just added transaction handler at ae9c56a @maoueh

@evaporei evaporei self-assigned this May 3, 2022
@evaporei
Copy link
Contributor

evaporei commented May 3, 2022

I just assigned myself to help on the review 👀

@evaporei
Copy link
Contributor

evaporei commented May 5, 2022

You might have to update tonic if this gets merged first.

evaporei
evaporei reviewed May 5, 2022
View reviewed changes
@clearloop
Copy link
Contributor Author

clearloop commented May 6, 2022

You might have to update tonic if this gets merged first.

updated

@clearloop clearloop requested review from evaporei and maoueh May 6, 2022 05:16
@evaporei
Copy link
Contributor

evaporei commented May 6, 2022

@clearloop the formatting is failing, you can add a #[rustfmt::skip] in the proto files.

@clearloop
Copy link
Contributor Author

clearloop commented May 6, 2022 •
edited
Loading

@clearloop the formatting is failing, you can add a #[rustfmt::skip] in the proto files.

just added #[rustfmt::skip] on the proto mod, and it works on my local machine, plz rerun the ci @evaporei

@clearloop clearloop mentioned this pull request May 6, 2022
Closed
@evaporei evaporei dismissed maoueh’s stale review May 11, 2022 22:37

review addressed

@evaporei evaporei merged commit b8faa45 into graphprotocol:master May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maoueh maoueh Awaiting requested review from maoueh

1 more reviewer

@evaporei evaporei evaporei approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@evaporei evaporei

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@clearloop @evaporei @maoueh