updated node version to v10 and removed v8 profiler (#25)

services/spelling/.nvmrc

@@ -1 +1 @@
-6.16.0
+10.16.0

services/spelling/Dockerfile

@@ -1,4 +1,4 @@
-FROM node:6.16.0 as app
+FROM node:10.16.0 as app
 
 WORKDIR /app
 
@@ -11,7 +11,7 @@ COPY . /app
 
 
 
-FROM node:6.16.0
+FROM node:10.16.0
 
 COPY --from=app /app /app
 

services/spelling/Jenkinsfile

@@ -36,13 +36,12 @@ pipeline {
       }
     }
 
-    // should be enabled once Node version is updated to >=8
-    // stage('Linting') {
-    //   steps {
-    //     sh 'DOCKER_COMPOSE_FLAGS="-f docker-compose.ci.yml" make format'
-    //     sh 'DOCKER_COMPOSE_FLAGS="-f docker-compose.ci.yml" make lint'
-    //   }
-    // }
+    stage('Linting') {
+      steps {
+        sh 'DOCKER_COMPOSE_FLAGS="-f docker-compose.ci.yml" make format'
+        sh 'DOCKER_COMPOSE_FLAGS="-f docker-compose.ci.yml" make lint'
+      }
+    }
 
     stage('Unit Tests') {
       steps {

services/spelling/app.js

@@ -34,16 +34,6 @@ server.get('/status', (req, res) => res.send({ status: 'spelling api is up' }))
 
 server.get('/health_check', HealthCheckController.healthCheck)
 
-const profiler = require('v8-profiler')
-server.get('/profile', function(req, res) {
-  const time = parseInt(req.query.time || '1000')
-  profiler.startProfiling('test')
-  return setTimeout(function() {
-    const profile = profiler.stopProfiling('test')
-    return res.json(profile)
-  }, time)
-})
-
 const host =
   __guard__(
     Settings.internal != null ? Settings.internal.spelling : undefined,

services/spelling/buildscript.txt

@@ -1,5 +1,5 @@
 spelling
---node-version=6.16.0
+--node-version=10.16.0
 --script-version=1.1.21
 --build-target=docker
 --dependencies=mongo,redis

services/spelling/package.json

@@ -36,8 +36,7 @@
     "redis": "~0.8.4",
     "request": "^2.53.0",
     "settings-sharelatex": "^1.1.0",
-    "underscore": "1.4.4",
-    "v8-profiler": "^5.2.4"
+    "underscore": "1.4.4"
   },
   "devDependencies": {
     "bunyan": "^1.0.0",