search

darwinia-network / shadow

Darwinia ZK proof services
GNU General Public License v3.0
2 stars 1 forks source link

Multistore unmerge #133

Closed wuminzhe closed 3 years ago

hackfisher commented 3 years ago

@xiaoch05 The tests after latest commit failed.

@wuminzhe I think this PR is still too large, need description documenting the main change sections.

xiaoch05 commented 3 years ago

@xiaoch05 The tests after latest commit failed.

@wuminzhe I think this PR is still too large, need description documenting the main change sections.

Yes, the package is split into several packages. So the test must be added for each package. The old test cases cannot be used anymore.

xiaoch05 commented 3 years ago

@xiaoch05 The tests after latest commit failed.

@wuminzhe I think this PR is still too large, need description documenting the main change sections.

@HackFisher This PR I have only tested import and runner. Other features such as mmr_root, ethashproof, receipt are not tested. So please wait to merge it until I have added test cases for each package, and tessted all the APIs.

wuminzhe commented 3 years ago

@HackFisher @xiaoch05 API response changed, pls pay more attention and read the new apis section of README.md