doc: use eslint

Makefile

@@ -90,6 +90,7 @@ cctest: all
 test: | cctest  # Depends on 'all'.
 	$(PYTHON) tools/test.py --mode=release message parallel sequential -J
 	$(MAKE) jslint
+	$(MAKE) jslint-docs
 	$(MAKE) cpplint
 
 test-parallel: all
@@ -519,6 +520,9 @@ bench-idle:
 jslint:
 	$(NODE) tools/eslint/bin/eslint.js lib src test tools/doc tools/eslint-rules \
 		--rulesdir tools/eslint-rules --quiet
+jslint-docs:
+	$(NODE) tools/eslint/bin/eslint.js --plugin eslint-plugin-markdown --ext markdown doc/api/ \
+		--rulesdir tools/eslint-rules --quiet
 
 CPPLINT_EXCLUDE ?=
 CPPLINT_EXCLUDE += src/node_lttng.cc
@@ -545,11 +549,11 @@ CPPLINT_FILES = $(filter-out $(CPPLINT_EXCLUDE), $(wildcard \
 cpplint:
 	@$(PYTHON) tools/cpplint.py $(CPPLINT_FILES)
 
-lint: jslint cpplint
+lint: jslint jslint-docs cpplint
 
-.PHONY: lint cpplint jslint bench clean docopen docclean doc dist distclean \
+.PHONY: lint cpplint jslint jslint-docs bench clean docopen docclean doc \
 	check uninstall install install-includes install-bin all staticlib \
 	dynamiclib test test-all test-addons build-addons website-upload pkg \
 	blog blogclean tar binary release-only bench-http-simple bench-idle \
 	bench-all bench bench-misc bench-array bench-buffer bench-net \
-	bench-http bench-fs bench-tls cctest run-ci
+	bench-http bench-fs bench-tls cctest run-ci distclean dist

doc/api/.eslintrc

@@ -0,0 +1,5 @@
+## Documentation-specific linter rules
+
+rules:
+  strict: 0
+  eol-last: 0

doc/api/addons.markdown

@@ -318,7 +318,7 @@ Once compiled, the example Addon can be required and used from within Node.js:
 // test.js
 const addon = require('./build/Release/addon');
 
-console.log('This should be eight:', addon.add(3,5));
+console.log('This should be eight:', addon.add(3, 5));
 ```
 
 
@@ -423,7 +423,7 @@ const addon = require('./build/Release/addon');
 
 var obj1 = addon('hello');
 var obj2 = addon('world');
-console.log(obj1.msg+' '+obj2.msg); // 'hello world'
+console.log(obj1.msg + ' ' + obj2.msg); // 'hello world'
 ```
 
 
@@ -638,9 +638,9 @@ Test it with:
 const addon = require('./build/Release/addon');
 
 var obj = new addon.MyObject(10);
-console.log( obj.plusOne() ); // 11
-console.log( obj.plusOne() ); // 12
-console.log( obj.plusOne() ); // 13
+console.log(obj.plusOne()); // 11
+console.log(obj.plusOne()); // 12
+console.log(obj.plusOne()); // 13
 ```
 
 ### Factory of wrapped objects
@@ -649,6 +649,7 @@ Alternatively, it is possible to use a factory pattern to avoid explicitly
 creating object instances using the JavaScript `new` operator:
 
 ```js
+/* eslint no-unused-vars:0, no-undef:0 */
 var obj = addon.createObject();
 // instead of:
 // var obj = new addon.Object();
@@ -824,14 +825,14 @@ Test it with:
 const createObject = require('./build/Release/addon');
 
 var obj = createObject(10);
-console.log( obj.plusOne() ); // 11
-console.log( obj.plusOne() ); // 12
-console.log( obj.plusOne() ); // 13
+console.log(obj.plusOne()); // 11
+console.log(obj.plusOne()); // 12
+console.log(obj.plusOne()); // 13
 
 var obj2 = createObject(20);
-console.log( obj2.plusOne() ); // 21
-console.log( obj2.plusOne() ); // 22
-console.log( obj2.plusOne() ); // 23
+console.log(obj2.plusOne()); // 21
+console.log(obj2.plusOne()); // 22
+console.log(obj2.plusOne()); // 23
 ```
 
 
@@ -1075,6 +1076,7 @@ Test in JavaScript by running:
 
 ```js
 // test.js
+/* eslint no-unused-vars:0 */
 const addon = require('./build/Release/addon');
 ```
 

doc/api/assert.markdown

@@ -41,6 +41,8 @@ results. For example, the following example does not throw an `AssertionError`
 because the properties on the [`Error`][] object are non-enumerable:
 
 ```js
+const assert = require('assert');
+
 // WARNING: This does not throw an AssertionError!
 assert.deepEqual(Error('a'), Error('b'));
 ```
@@ -65,7 +67,7 @@ const obj3 = {
   a : {
     b : 1
   }
-}
+};
 const obj4 = Object.create(obj1);
 
 assert.deepEqual(obj1, obj1);
@@ -124,6 +126,8 @@ The following, for instance, will throw the [`TypeError`][] because there is no
 matching error type in the assertion:
 
 ```js
+const assert = require('assert');
+
 assert.doesNotThrow(
   () => {
     throw new TypeError('Wrong value');
@@ -136,6 +140,8 @@ However, the following will result in an `AssertionError` with the message
 'Got unwanted exception (TypeError)..':
 
 ```js
+const assert = require('assert');
+
 assert.doesNotThrow(
   () => {
     throw new TypeError('Wrong value');
@@ -149,6 +155,8 @@ parameter, the value of `message` will be appended to the `AssertionError`
 message:
 
 ```js
+const assert = require('assert');
+
 assert.doesNotThrow(
   () => {
     throw new TypeError('Wrong value');
@@ -208,7 +216,7 @@ const assert = require('assert');
 
 assert.ifError(0); // OK
 assert.ifError(1); // Throws 1
-assert.ifError('error') // Throws 'error'
+assert.ifError('error'); // Throws 'error'
 assert.ifError(new Error()); // Throws Error
 ```
 
@@ -233,7 +241,7 @@ const obj3 = {
   a : {
     b : 1
   }
-}
+};
 const obj4 = Object.create(obj1);
 
 assert.notDeepEqual(obj1, obj1);
@@ -366,6 +374,8 @@ constructor, [`RegExp`][], or validation function.
 Validate instanceof using constructor:
 
 ```js
+const assert = require('assert');
+
 assert.throws(
   () => {
     throw new Error('Wrong value');
@@ -377,6 +387,8 @@ assert.throws(
 Validate error message using [`RegExp`][]:
 
 ```js
+const assert = require('assert');
+
 assert.throws(
   () => {
     throw new Error('Wrong value');
@@ -388,12 +400,14 @@ assert.throws(
 Custom error validation:
 
 ```js
+const assert = require('assert');
+
 assert.throws(
   () => {
     throw new Error('Wrong value');
   },
   function(err) {
-    if ( (err instanceof Error) && /value/.test(err) ) {
+    if ((err instanceof Error) && /value/.test(err)) {
       return true;
     }
   },