This repository has been archived by the owner on Jul 28, 2021. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 155
Documentation review guidelines
Prasad Ramesh edited this page May 13, 2021
·
3 revisions
- The documentation should explain the working of a feature thoroughly
- Often the paragraphs and definitions lack clarity, it should be clear and comprehensive enough for new users to understand it
- All "features" i.e., non-mandatory fields should also be explained in the documentation
- If something can be seen on the screen, its features should be documented and explained
- Run the text across a grammar and spellcheck tool like Grammarly.
- Check for and correct any errors.
- Should in English US, not UK. Set system language and Grammarly language to English US to show correct suggestions.
- Should be active voice and not passive
- All screenshots and examples should be in USD currency ($)
- Should have little padding on either side
- Effect should same as achieved on a Macbook Air with screen resolution set at 1280x800
- Screenshots should be placed in respective appropriate folders
- Subfolders under a module for screenshots are avoided for easier upgrades and maintenance
- Check if all image URL paths in the text also have corresponding image files attached in the PR, easier way for large PRs is to pull it locally and check
- When a new page is added, check whether the PR also contains index listing in contents.md for the module