Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

lint the markdown #3172

Closed
boneskull opened this issue Dec 21, 2017 · 1 comment · Fixed by #3173 or Urigo/tortilla#64
Closed

lint the markdown #3172

boneskull opened this issue Dec 21, 2017 · 1 comment · Fixed by #3173 or Urigo/tortilla#64
Assignees
@boneskull
Labels
area: documentation anything involving docs or mochajs.org

Comments

@boneskull
Copy link
Contributor

We have enough markdown in here that it should be linted via markdownlint.
@boneskull boneskull added area: documentation anything involving docs or mochajs.org qa labels Dec 21, 2017
@boneskull boneskull self-assigned this Dec 21, 2017
@Bamieh
Copy link
Contributor

Bamieh commented Dec 21, 2017

@boneskull good idea. Are you going to work on this or can i get started with it?

boneskull added a commit that referenced this issue Dec 21, 2017
@boneskull
add @mocha/markdownlint-cli; closes #3172
9f35b56 
- forked [markdownlint-cli](https://npm.im/markdownlint-cli) to fix a
  glob-related bug; will abandon our version if/when it's merged
- add `.markdownlint.json` config
- remove "allow trailing spaces" for Markdown in `.editorconfig`
- add to `lint` target & create `markdownlint` script in `package.json`,
  which is dumb, but we're stuck with it for now
boneskull added a commit that referenced this issue Dec 21, 2017
@boneskull
add @mocha/markdownlint-cli; closes #3172
0aae24e 
- forked [markdownlint-cli](https://npm.im/markdownlint-cli) to fix a
  glob-related bug; will abandon our version if/when it's merged
- add `.markdownlint.json` config
- remove "allow trailing spaces" for Markdown in `.editorconfig`
- add to `lint` target & create `markdownlint` script in `package.json`,
  which is dumb, but we're stuck with it for now (+1 squashed commit)
Squashed commits:
[baf6387] wip
boneskull added a commit that referenced this issue Dec 21, 2017
@boneskull
add @mocha/markdownlint-cli; closes #3172
c240edc 
- forked [markdownlint-cli](https://npm.im/markdownlint-cli) to fix a
  glob-related bug; will abandon our version if/when it's merged
- add `.markdownlint.json` config
- remove "allow trailing spaces" for Markdown in `.editorconfig`
- add to `lint` target & create `markdownlint` script in `package.json`,
  which is dumb, but we're stuck with it for now
@boneskull boneskull mentioned this issue Dec 21, 2017
Merged
boneskull added a commit that referenced this issue Dec 28, 2017
@boneskull
add @mocha/markdownlint-cli; closes #3172
98a8313 
- forked [markdownlint-cli](https://npm.im/markdownlint-cli) to fix a
  glob-related bug; will abandon our version if/when it's merged
- add `.markdownlint.json` config
- remove "allow trailing spaces" for Markdown in `.editorconfig`
- add to `lint` target & create `markdownlint` script in `package.json`,
  which is dumb, but we're stuck with it for now
boneskull added a commit that referenced this issue Dec 29, 2017
@boneskull
add @mocha/markdownlint-cli; closes #3172
564b681 
- forked [markdownlint-cli](https://npm.im/markdownlint-cli) to fix a
  glob-related bug; will abandon our version if/when it's merged
- add `.markdownlint.json` config
- remove "allow trailing spaces" for Markdown in `.editorconfig`
- add to `lint` target & create `markdownlint` script in `package.json`,
  which is dumb, but we're stuck with it for now
boneskull added a commit that referenced this issue Dec 29, 2017
@boneskull
add @mocha/markdownlint-cli; closes #3172
c22aad8 
- forked [markdownlint-cli](https://npm.im/markdownlint-cli) to fix a
  glob-related bug; will abandon our version if/when it's merged
- add `.markdownlint.json` config
- remove "allow trailing spaces" for Markdown in `.editorconfig`
- add to `lint` target & create `markdownlint` script in `package.json`,
  which is dumb, but we're stuck with it for now
boneskull added a commit that referenced this issue Jan 2, 2018
@boneskull
add @mocha/markdownlint-cli; closes #3172
01f000f 
- forked [markdownlint-cli](https://npm.im/markdownlint-cli) to fix a
  glob-related bug; will abandon our version if/when it's merged
- add `.markdownlint.json` config
- remove "allow trailing spaces" for Markdown in `.editorconfig`
- add to `lint` target & create `markdownlint` script in `package.json`,
  which is dumb, but we're stuck with it for now
boneskull added a commit that referenced this issue Jan 3, 2018
@boneskull
lint Markdown; closes #3172
b61e91b 
- add [markdownlint-cli](https://npm.im/markdownlint-cli)
- lint all Markdown files
- add some rules to `.markdownlintrc`
boneskull added a commit that referenced this issue Jan 3, 2018
@boneskull
lint Markdown; closes #3172
a723b8f 
- add [markdownlint-cli](https://npm.im/markdownlint-cli)
- lint all Markdown files
- add some rules to `.markdownlintrc`
This was referenced Jan 18, 2018
Merged
Merged
@renovate renovate bot mentioned this issue Feb 14, 2018
Merged
This was referenced Sep 22, 2018
Closed
Closed
Closed
Open
Closed
Closed
Closed
Closed
Closed
Closed
Merged
Closed
This was referenced Sep 23, 2018
Closed
Merged
Closed
Closed
Closed
This was referenced Oct 2, 2018
Closed
Closed
Closed
Open
This was referenced Oct 28, 2018
Merged
Closed
Closed
Merged
This was referenced Nov 4, 2018
Merged
Merged
Merged
This was referenced Nov 12, 2018
Closed
Merged
Closed
@renovate renovate bot mentioned this issue Nov 23, 2018
Merged
This was referenced Dec 1, 2018
Closed
Merged
Merged
This was referenced Dec 8, 2018
Merged
Closed
Merged
Closed
This was referenced Dec 23, 2018
Closed
Closed
sgilroy pushed a commit to TwineHealth/mocha that referenced this issue Feb 27, 2019
@boneskull
lint Markdown; closes mochajs#3172
150185a 
- add [markdownlint-cli](https://npm.im/markdownlint-cli)
- lint all Markdown files
- add some rules to `.markdownlintrc`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@boneskull boneskull

Labels
area: documentation anything involving docs or mochajs.org
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

lint the markdown, closes #3172 mochajs/mocha
Update dependency mocha to v5 Urigo/tortilla
2 participants
@boneskull @Bamieh