Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Conversation

@gui1117
Copy link
Contributor

@gui1117 gui1117 commented Mar 12, 2019

related to #1917

It is not fully defined definitive doc but at least it makes a start for future improvment which will rely on modifying doc only.

Also I'm still mixed about something: Documentation passed in decl_module appears for Call enum which makes sense, but then Module structure has no documentation. it may needs none, I don't know what to write on it.

Now except Module struct all type of a module can be documented with macro or have default documentation.

@gui1117 gui1117 requested a review from bkchr March 12, 2019 14:03
@gui1117 gui1117 added A0-please_review Pull request needs code review. and removed A4-gotissues labels Mar 12, 2019
@bkchr
Copy link
Member

bkchr commented Mar 14, 2019

Please merge master.

@gui1117 gui1117 merged commit 8162aa3 into master Mar 14, 2019
@bkchr bkchr deleted the gui-doc branch March 14, 2019 17:16
MTDK1 pushed a commit to bdevux/substrate that referenced this pull request Apr 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants