Skip to content

feat(chain): Arweave integration#3465

Merged
evaporei merged 21 commits into
graphprotocol:masterfrom
ChainSafe:cs/arweave
May 11, 2022
Merged

feat(chain): Arweave integration#3465
evaporei merged 21 commits into
graphprotocol:masterfrom
ChainSafe:cs/arweave

Conversation

@clearloop

@clearloop clearloop commented Apr 13, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor

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
Comment thread chain/arweave/src/adapter.rs Outdated
Comment thread chain/arweave/src/chain.rs Outdated
Comment thread chain/arweave/src/data_source.rs Outdated
Comment thread chain/arweave/src/data_source.rs Outdated
Comment thread chain/arweave/src/data_source.rs Outdated
Comment thread chain/arweave/src/protobuf/google.protobuf.rs Outdated
Comment thread chain/arweave/src/runtime/abi.rs
Comment thread graph/src/runtime/mod.rs Outdated
Comment thread node/src/main.rs Outdated
@clearloop

clearloop commented Apr 28, 2022

Copy link
Copy Markdown
Contributor Author

Just added transaction handler at ae9c56a @maoueh

@evaporei evaporei self-assigned this May 3, 2022
@evaporei

evaporei commented May 3, 2022

Copy link
Copy Markdown
Contributor

I just assigned myself to help on the review 👀

@evaporei

evaporei commented May 5, 2022

Copy link
Copy Markdown
Contributor

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

evaporei
evaporei reviewed May 5, 2022
View reviewed changes
Comment thread chain/arweave/src/runtime/generated.rs Outdated
Comment thread chain/arweave/src/runtime/generated.rs
@clearloop

clearloop commented May 6, 2022

Copy link
Copy Markdown
Contributor Author

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

evaporei commented May 6, 2022

Copy link
Copy Markdown
Contributor

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

@clearloop

clearloop commented May 6, 2022 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

@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