Merge pull request #19 from pbredenberg/chore-upgrade-node

Upgrade Node and add standards enforcement

.markdownlint.json

@@ -0,0 +1,3 @@
+{
+   "extends": "./node_modules/@silvermine/standardization/.markdownlint.json"
+}

.nvmrc

@@ -1 +1 @@
-8.10.0
+12.14.0

.travis.yml

@@ -2,11 +2,18 @@ language: node_js
 node_js:
    - "node" # Latest node version
    - "lts/*" # Latest LTS version
+   - "12"
    - "10"
    - "8"
-   - "8.10"
 
-before_install: if [[ `npm -v` != 6* ]]; then npm i -g [email protected]; fi
+before_install: npm i -g [email protected]
+
+script:
+   - node --version
+   - npm --version
+   - commitlint-travis
+   - grunt standards
+   - npm test
 
 # For code coverage:
 after_success:

Gruntfile.js

@@ -1,5 +1,7 @@
 'use strict';
 
+var markdownlint = require('markdownlint');
+
 module.exports = function(grunt) {
 
    var config;
@@ -18,11 +20,22 @@ module.exports = function(grunt) {
          target: config.js.all,
       },
 
+      markdownlint: {
+         all: {
+            src: [ 'README.md' ],
+            options: {
+               // eslint-disable-next-line no-sync
+               config: markdownlint.readConfigSync('.markdownlint.json'),
+            },
+         },
+      },
+
    });
 
    grunt.loadNpmTasks('grunt-eslint');
+   grunt.loadNpmTasks('grunt-markdownlint');
 
-   grunt.registerTask('standards', [ 'eslint' ]);
+   grunt.registerTask('standards', [ 'eslint', 'markdownlint' ]);
    grunt.registerTask('default', [ 'standards' ]);
 
 };

README.md

@@ -29,7 +29,7 @@ NOTE: at this time, it is assumed that the topic is in the same account and
 region as the Lambda function itself. That can be changed in the future if
 needed - feel free to open an issue, and preferably submit a pull request.
 
-```
+```yaml
 functions:
    doSomething:
       name: ${self:service}-${self:provider.stage}-doSomething

commitlint.config.js

@@ -0,0 +1,5 @@
+'use strict';
+
+module.exports = {
+   extends: [ '@silvermine/standardization/commitlint.js' ],
+};