Merge pull request #431 from adamtheturtle/combine-lint

Combine lint

Author: adamtheturtle

.travis.yml

@@ -49,7 +49,6 @@ install:
 cache: pip
 before_script:
 - make lint
-- markdownlint --config .markdownlint.json README.md CONTRIBUTING.md
 - python ci/set_secrets_file.py
 script:
 - pytest --exitfirst -vvv tests/mock_vws/"$TEST_FILENAME" --cov=src --cov=tests

CONTRIBUTING.md

@@ -6,12 +6,18 @@ Travis CI is the canonical source truth.
 
 ## Install Contribution Dependencies
 
-Install dependencies in a virtual environment.
+Install Python dependencies in a virtual environment.
 
 ```sh
 pip install --editable .[dev]
 ```
 
+Insall `npm` dependencies for linting Markdown files:
+
+```
+npm install -g markdownlint-cli
+```
+
 Spell checking requires `enchant`.
 This can be installed on macOS, for example, with [Homebrew](http://brew.sh):
 
@@ -25,6 +31,8 @@ and on Ubuntu with `apt`:
 apt-get install -y enchant
 ```
 
+and to lint
+
 ## Linting
 
 Run lint tools:

Makefile

@@ -14,6 +14,7 @@ lint:
 	pyroma .
 	vulture . --min-confidence 100
 	yapf --diff --recursive src/ tests/ ci/
+	markdownlint --config .markdownlint.json README.md CONTRIBUTING.md
 
 fix-lint:
 	autoflake --in-place --recursive --remove-all-unused-imports --remove-unused-variables .