Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

A0-3132: Add pallets documentation to metadata #1403

Merged
merged 4 commits into from
Sep 19, 2023
Merged

A0-3132: Add pallets documentation to metadata #1403

merged 4 commits into from
Sep 19, 2023

Conversation

Marcin-Radecki
Copy link
Contributor

Description

paritytech/substrate#13452 adds the possibility to include readme.md (markdown) docs into pallet metadata. This is not yet supported in our current metadata, yet it is a preparation in case metadata V15 lands on our chain.

Also, this PR slightly reduces docs duplication by extracting our pallet file docs to README.md. That should be beneficial to a reader who browse pallet code in GH.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

Testing

  • run cargo doc for our pallets

@Marcin-Radecki Marcin-Radecki added this pull request to the merge queue Sep 19, 2023
Merged via the queue into main with commit b14a20f Sep 19, 2023
@Marcin-Radecki Marcin-Radecki deleted the A0-3132 branch September 19, 2023 08:32
fixxxedpoint pushed a commit that referenced this pull request Sep 25, 2023
# Description

paritytech/substrate#13452 adds the possibility
to include readme.md (markdown) docs into pallet metadata. This is not
yet supported in our current metadata, yet it is a preparation in case
metadata V15 lands on our chain.

Also, this PR slightly reduces docs duplication by extracting our pallet
file docs to README.md. That should be beneficial to a reader who browse
pallet code in GH.

## Type of change

Please delete options that are not relevant.

- New feature (non-breaking change which adds functionality)

## Testing

- run `cargo doc` for our pallets
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants