Merge pull request #239 from Financial-Times/upgrade-to-node-v12

Upgrade to Node v12

.circleci/config.yml

@@ -3,10 +3,10 @@
 
 references:
 
-  container_config_node8: &container_config_node8
+  container_config_node12: &container_config_node12
     working_directory: ~/project/build
     docker:
-      - image: circleci/node:8-browsers
+      - image: circleci/node:12-browsers
 
   workspace_root: &workspace_root
     ~/project
@@ -61,7 +61,7 @@ version: 2
 jobs:
 
   build:
-    <<: *container_config_node8
+    <<: *container_config_node12
     steps:
       - checkout
       - run:
@@ -91,7 +91,7 @@ jobs:
             - build
 
   test:
-    <<: *container_config_node8
+    <<: *container_config_node12
     steps:
       - *attach_workspace
       - run:
@@ -107,7 +107,7 @@ jobs:
           destination: test-results
 
   publish:
-    <<: *container_config_node8
+    <<: *container_config_node12
     steps:
       - *attach_workspace
       - run:

.nvmrc

@@ -0,0 +1 @@
+12

README.md

@@ -6,6 +6,7 @@
 ## Requirements
 
 * Node 8.x
+* Node version defined by `.nvmrc` (run command `nvm use` to switch your local Node version to the one in this file).
 * [Vault](https://github.com/Financial-Times/vault/wiki/Getting-Started-With-Vault)