sensu-docs icon indicating copy to clipboard operation
sensu-docs copied to clipboard

Standardize use of whitespace in markdown files

Open apaskulin opened this issue 6 years ago • 5 comments

This is an issue with:

  • [x] Bug (site functionality or styling)
  • [ ] Errata (fix needed for doc content)
  • [ ] New content (guide wanted)
  • [ ] Update (Add missing or refresh existing content)

Expected Behavior

Easy-to-review PRs and consistent whitespace formatting throughout the docs.

Note: Markdown tables in Hugo require a trailing space in the first line

Current Behavior

In some cases, editors are automatically changing line endings and other whitespace formatting in the markdown files, which makes for confusing PRs (see #381).

Possible Solution

apaskulin avatar Jun 12 '18 16:06 apaskulin

:+1: FWIW, I use markdownlint

asachs01 avatar Jun 13 '18 18:06 asachs01

Noting that tables are ok with a closing pipe in the first line, possibly helping us with using markdownlint:

to           |     |
-------------|------
description  | Content

apaskulin avatar Jun 14 '18 17:06 apaskulin

I think Vale might cover Markdown syntax and style rules: https://github.com/errata-ai/vale/

hillaryfraley avatar Dec 03 '20 14:12 hillaryfraley

Hillary to follow Vale documentation to use locally to confirm whether we can customize with our style guide. I'm not sure Vale will lint for Markdown syntax and whitespace standardization.

hillaryfraley avatar Feb 11 '21 21:02 hillaryfraley

I don't think Vale will work for our purposes. It seems to be a style linter.

hillaryfraley avatar Mar 23 '21 18:03 hillaryfraley