Markdown.md

Markdown Code Reviews

Style Guide

CSE developers treat documentation like other source code and follow the same rules and checklists when reviewing documentation as code.

Documentation should both use good markdown syntax to ensure it's properly parsed, and follow good writing style guidelines to ensure the document is easy to read and understand.

Code Analysis / Linting

Linting tools exist both for verifying proper markdown syntax as well as grammar and proper English language.

A good setup includes a markdown linter used during editing and PR build verification, and a grammar linter used while editing the document. The following are a list of linters that could be used in this setup.

markdownlint

markdownlint is a linter for markdown that verifies markdown syntax, and also enforces rules that make the text more readable.

It's available as a ruby gem, an npm package, a Node.js CLI and a VS Code extension.

Installing the Node.js CLI

npm install -g markdownlint-cli

Running markdownlint on a Node.js project

markdownlint **/*.md --ignore node_modules

Fixing errors automatically

markdownlint **/*.md --ignore node_modules --fix

proselint

proselint is a command line utility that lints the text contents of the document. It checks for jargon, spelling errors, redundancy, corporate speak and other language related issues.

It's available both as a python package and a node package.

pip install proselint
npm install -g proselint

Run proselint

proselint document.md

write-good

write-good is a linter for English text that helps writing better documentation.

npm install -g write-good

Run write-good

write-good *.md

Run write-good without installing it

npx write-good *.md

Write Good is also available as an extension for VS Code

VS Code Extensions

Write Good Linter

The Write Good Linter Extension integrates with VS Code to give grammar and language advice while editing the document.

markdownlint extension

The markdownlint extension examines the markdown documents, showing warnings for rule violations while editing.

Build validation

To automate linting with markdownlint for PR validation in GitHub actions as we do in this repo, use the following YAML.

name: Markdownlint

on:
  push:
    paths:
      - "**/*.md"
  pull_request:
    paths:
      - "**/*.md"

jobs:
  lint:

    runs-on: ubuntu-latest

    steps:
    - uses: actions/checkout@v2
    - name: Use Node.js
      uses: actions/setup-node@v1
      with:
        node-version: 12.x
    - name: Run Markdownlint
      run: |
        npm i -g markdownlint-cli
        markdownlint "**/*.md" --ignore node_modules

Code Review Checklist

Apart from the items on the Code Review Checklist you should also look for these markdown specific code review items

• Is the document easy to read and understand and does it follow good writing guidelines?
• Is there a single source of truth or is content repeated in more than one document?
• Is the documentation up to date with the code?
• Is the documentation technically, and ethically correct?

Writing Style Guidelines

The following are some examples of writing style guidelines.

Agree in your team which guidelines you should apply to your project documentation. Save your guidelines together with your documentation so they are easy to refer back to.

Wording

• Use inclusive language, and avoid jargon and uncommon words. The docs should be easy to understand
• Be clear and concise, stick to the goal of the document
• Use active voice
• Spell check and grammar check the text
• Always follow chronological order

Document Organization

• Organize documents by topic rather than type, this makes it easier to find the documentation
• Each folder should have a top-level readme.md and any other documents within that folder should link directly or indirectly from that readme.md
• Document names with more than one word should use underscores instead of spaces, for example machine_learning_pipeline_design.md. The same applies to images

Headings

• Start with a H1 (single # in markdown) and respect the order H1 > H2 > H3 etc
• Follow each heading with text before proceeding with the next heading
• Avoid putting numbers in headings. Numbers shift, and can create outdated titles
• Avoid using symbols and special characters in headers, this causes problems with anchor links
• Avoid links in headers

Links

• Avoid duplication of content, instead link to the single source of truth
• Link but don't summarize. Summarizing content on another page leads to the content living in two places
• Use meaningful anchor texts, e.g. instead of writing Follow the instructions [here](../recipes/Markdown.md) write Follow the [Markdown guidelines](../recipes/Markdown.md)

Lists

• List items should start with capital letters if possible
• Use ordered lists when the items describe a sequence to follow, otherwise use unordered lists
• For ordered lists, prefix each item with 1. When rendered, the list items will appear with sequential numbering. This avoids number-gaps in list
• Do not add commas (,) or semicolons (;) to the end of list items, and avoid periods (.) unless the list item represents a complete sentence

Images

• Place images in a separate directory named img
• Name images appropriately, avoiding generic names like screenshot.png
• Avoid adding large images or videos to source control, link to an external location instead

Emphasis and special sections

• Use bold or italic to emphasise

  For sections that everyone reading this document needs to be aware of, use blocks

• Use backticks for code, a single backtick for inline code like pip install flake8 and 3 backticks for codeblocks followed by the language for syntax highlighting

  def add(num1: int, num2: int):
    return num1 + num2

• Use check boxes for task lists

  • Item 1
  • Item 2
  • Item 3

• Add a References section to the end of the document with links to external references

• Prefer tables over lists for comparisons and reports to make research and results more readable

  Option	Pros	Cons
  Option 1	Some pros	Some cons
  Option 2	Some pros	Some cons

General

• Always use markdown syntax, don't mix with HTML