search

regen-network / cosmos-modules

Other
10 stars 2 forks source link

Implement Proposal execution #57

Closed alpe closed 4 years ago

alpe commented 4 years ago

After #66

Resolves #38

alpe commented 4 years ago

As with the other PRs there is no "proper documentation" written. I have tried to define test cases for all the scenarios I could imagine so that they are a specification as code. Probably not what you were looking for. I think we should make documentation a first class task. But before, I would appreciate if we can be more precise what kind of documentation, where we host it and who we are targeting as readers. Otherwise it would become an epic. We currently have only https://github.com/regen-network/cosmos-modules/issues/40 which is not on the critical path.

The testdata messages are used for the tests in ProposalExecutor and Keeper. I see them also as extension point so that people may use them for their test scenarios.

aaronc commented 4 years ago

@clevinson can you comment on documentation here?

alpe commented 4 years ago

I have created an issue and linked to the comment