Skip to content
This repository has been archived by the owner on Aug 19, 2020. It is now read-only.

Update floats.md #85

Open
wants to merge 2 commits into
base: apopiak-floats
Choose a base branch
from
Open

Update floats.md #85

wants to merge 2 commits into from

Conversation

wheresaddie
Copy link
Contributor

Floating Point in Substrate explained

Thank you for making a PR to the knowledge base. Please make sure it follows these short guidelines:

  • Are the audience and objective of the document clear?
  • Is the writing:
    • Clear: All jargon defined, explained, or linked to. Best to avoid it.
    • Precise: No ambiguous meanings
    • Concise: Free of superfluous detail
  • Does it follow our style guide?
  • If this is a new page, does the PR include the appropriate infrastructure, e.g. adding the
    page to a sidebar?
  • Build the page. Does it render properly? E.g. no funny lists or formatting.
  • Links should go to rustdocs or devhub articles, not source code.
  • If this PR addresses an issue in the queue, have you referenced it in the description?

### Floating Point in Substrate explained
@wheresaddie wheresaddie requested a review from apopiak June 29, 2020 13:29
current/runtime/floats.md Show resolved Hide resolved
current/runtime/floats.md Outdated Show resolved Hide resolved
going to add an additional explanation of general/transferable reasoning after 'Perbill'/ 'Permill' for next PR

Co-authored-by: Alexander Popiak <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants