Skip to content
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

feat(rosettarpc): Data API implementation #2940

Merged
merged 19 commits into from
Aug 31, 2020

Conversation

frol
Copy link
Collaborator

@frol frol commented Jul 2, 2020

Resolves #2738

The spec is: https://raw.githubusercontent.com/coinbase/rosetta-specifications/master/api.json (you can view it on https://petstore.swagger.io/)

README

This implementation is battle-tested and checked by Coinbase using automated tooling (rosetta-cli)

@gitpod-io
Copy link

gitpod-io bot commented Jul 2, 2020

chain/rosettarpc/src/lib.rs Outdated Show resolved Hide resolved
chain/rosettarpc/src/lib.rs Outdated Show resolved Hide resolved
neard/src/lib.rs Outdated Show resolved Hide resolved
@frol frol force-pushed the feature/rosetta-api branch 2 times, most recently from 43c8d04 to d04dab9 Compare August 13, 2020 20:11
@frol frol marked this pull request as ready for review August 13, 2020 21:22
@frol frol changed the title feat(rosettarpc): initial implementation (WIP) feat(rosettarpc): Data API implementation Aug 13, 2020
@codecov
Copy link

codecov bot commented Aug 13, 2020

Codecov Report

Merging #2940 into master will increase coverage by 0.12%.
The diff coverage is 73.91%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2940      +/-   ##
==========================================
+ Coverage   87.02%   87.15%   +0.12%     
==========================================
  Files         216      216              
  Lines       42119    42123       +4     
==========================================
+ Hits        36655    36712      +57     
+ Misses       5464     5411      -53     
Impacted Files Coverage Δ
core/primitives/src/types.rs 44.22% <ø> (-1.51%) ⬇️
neard/src/config.rs 94.32% <ø> (ø)
neard/src/lib.rs 60.36% <ø> (ø)
runtime/runtime/src/actions.rs 79.61% <ø> (+0.76%) ⬆️
runtime/runtime/src/verifier.rs 93.55% <0.00%> (-0.12%) ⬇️
core/runtime-configs/src/lib.rs 89.13% <73.33%> (-10.87%) ⬇️
core/primitives/src/views.rs 58.30% <100.00%> (+0.20%) ⬆️
runtime/runtime/src/lib.rs 87.92% <100.00%> (+0.01%) ⬆️
core/store/src/trie/state_parts.rs 95.77% <0.00%> (-1.41%) ⬇️
... and 14 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7cf0fad...384e64f. Read the comment docs.

@frol frol requested a review from bowenwang1996 August 18, 2020 16:22
chain/rosetta-rpc/README.md Outdated Show resolved Hide resolved
chain/rosetta-rpc/src/lib.rs Outdated Show resolved Hide resolved
chain/rosetta-rpc/src/models.rs Outdated Show resolved Hide resolved
@frol frol force-pushed the feature/rosetta-api branch from 03787c8 to 13a8284 Compare August 20, 2020 15:34
chain/rosetta-rpc/src/adapters.rs Outdated Show resolved Hide resolved
chain/rosetta-rpc/src/adapters.rs Outdated Show resolved Hide resolved
chain/rosetta-rpc/src/adapters.rs Outdated Show resolved Hide resolved
chain/rosetta-rpc/src/adapters.rs Outdated Show resolved Hide resolved
@frol frol added the automerge label Aug 31, 2020
@nearprotocol-bulldozer nearprotocol-bulldozer bot merged commit 4e89617 into master Aug 31, 2020
@nearprotocol-bulldozer nearprotocol-bulldozer bot deleted the feature/rosetta-api branch August 31, 2020 16:07
SkidanovAlex added a commit that referenced this pull request Aug 31, 2020
SkidanovAlex added a commit that referenced this pull request Aug 31, 2020
frol added a commit that referenced this pull request Sep 2, 2020
mhalambek pushed a commit that referenced this pull request Sep 7, 2020
chefsale pushed a commit that referenced this pull request Sep 7, 2020
chefsale pushed a commit that referenced this pull request Sep 7, 2020
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.

Implement Rosetta RPC
3 participants