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

☂️ Formalize Pallet and Call docs #13299

Closed
ggwpez opened this issue Feb 2, 2023 · 8 comments · Fixed by #14101
Closed

☂️ Formalize Pallet and Call docs #13299

ggwpez opened this issue Feb 2, 2023 · 8 comments · Fixed by #14101
Labels
I6-documentation Documentation needs fixing, improving or augmenting. U3-nice_to_have Issue is worth doing eventually. Z3-substantial Can be fixed by an experienced coder with a working knowledge of the codebase.

Comments

@ggwpez
Copy link
Member

ggwpez commented Feb 2, 2023

The FRAME team realized that Pallet and pallet Call documentation should be more formalized and probably have some templates to them.
Ultimately this can be CI checked by parsing the docs with a macro and comparing against the templates.
This should help to have a unified basis to build higher-level docs upon

First step would be to agree on a structure for the pallet level docs and README requirements for them.

(cc @lisa-parity in case you have some ideas)

@ggwpez ggwpez added I6-documentation Documentation needs fixing, improving or augmenting. U3-nice_to_have Issue is worth doing eventually. Z3-substantial Can be fixed by an experienced coder with a working knowledge of the codebase. labels Feb 2, 2023
@lisa-parity
Copy link
Contributor

@ggwpez, this is really cool. I do have some ideas about this. I'll try. to sketch out a sample template next week.

@bkchr
Copy link
Member

bkchr commented Feb 6, 2023

CC paritytech/frame-metadata#47

@kianenigma
Copy link
Contributor

A step toward this is #13341

Right now, we have a section in the docs of some pallet that repeats the dispatchables. This should be just replaced with these modules.

@lisa-parity
Copy link
Contributor

At a high level, it would be nice for every pallet to cover the following, where applicable:

  • Purpose (what it's for, why it's important, the use case or problem it addresses)
  • Specialized terminology (though it would be great if it wasn't required)
  • Public functions
  • Private functions
  • How it work diagram/description
  • Dependencies
  • Storage items
  • Events
  • Errors
  • Examples

I gather PR #13341 is about automating the collection of information.

@bkchr
Copy link
Member

bkchr commented Feb 15, 2023

  • Public functions

  • Private functions

  • Dependencies

  • Storage items

  • Events

  • Errors

All of these don't need to be in the "high level" docs. The high level docs, as the name indicate should be a on a high level. How does the pallet work, what is it doing etc. Basically all the other things you mentioned there!

@ggwpez
Copy link
Member Author

ggwpez commented May 3, 2023

I tried out some scheme for the MQ pallet, and it basically turned into these sections. I will re-use that for the TxPause and SafeMode pallet, so please propose improvements.

  • Abstract: One to three sentences about what the pallet does. Not how or why, just what.
  • Design goals: Outline all specific goals of this pallet. This helps with understanding why stuff was designed in a specific way.
  • Design: Explain how it is designed and why it was chosen to do so. Should reference the design goals and how they are achieved.
  • Scenarios: Code examples together with explanations on how the pallet will internally process the example code. (note: examples are missing from the MQ pallet docs)
  • Terminology

Diagrams would be nice, but so far i did not find a good way for ASCII diagrams or something suitable for rust doc.
Another thing missing would be the Purpose probably…

@kianenigma
Copy link
Contributor

kianenigma commented May 9, 2023

Diagrams would be nice, but so far i did not find a good way for ASCII diagrams or something suitable for rust doc.
Another thing missing would be the Purpose probably…

Have you seen mermaid? It will probably have a rendered for rust-docs.

@kianenigma
Copy link
Contributor

kianenigma commented May 9, 2023

This is my suggestion. The main alteration I have is to make the docs read from "high level" to "low level". So something like design and design goals should be all the way at the end, as this is a very low level implementation note: https://hackmd.io/4pjRqPNNTnGXJHKK9jv90A

So at a high level, it is 3 steps:

  1. High Level / End-User Details (for those with least amount of knowledge)
  2. Code Details (mostly for external devs, wanting to interface with the pallet, DApp builders)
  3. Low Level / Implementation Details: for us, for those who want to get really deep.

@juangirini juangirini moved this from Backlog to In Progress in Runtime / FRAME May 23, 2023
@github-project-automation github-project-automation bot moved this from In Progress to Done in Runtime / FRAME May 29, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
I6-documentation Documentation needs fixing, improving or augmenting. U3-nice_to_have Issue is worth doing eventually. Z3-substantial Can be fixed by an experienced coder with a working knowledge of the codebase.
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

4 participants