Merge pull request #26 from opentable/GP-361

[GP-361] Fix rlp-email-templates integration, allow to run Mercury in dev mode again.

Author: federicomaffei

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "mercury-bot",
-  "version": "3.0.3",
+  "version": "3.1.0",
   "description": "Mercury is a bot for managing in-code static translations",
   "main": "src/app.js",
   "scripts": {
@@ -45,7 +45,7 @@
     "sinon": "^7.4.2"
   },
   "dependencies": {
-    "@octokit/rest": "16.28.9",
+    "@octokit/rest": "^16.43.2",
     "async": "2.6.3",
     "base-64": "0.1.0",
     "config": "1.31.0",

package.json

@@ -1,5 +1,7 @@
 'use strict';
 
+const MAX_ALLOWED_BODY_CHARS = 65536;
+
 const errorTypes = require('../constants/error-types');
 const metadataFormatter = require('../utils/format-pr-metadata');
 
@@ -24,7 +26,7 @@ module.exports = ({ emitter, config }) => (repository, callback) => {
     head: `${config.github.owner}:${config.github.branch}`,
     title: pullRequestMetadata.title,
     base: repository.manifestContent.workingBranch,
-    body: pullRequestMetadata.body
+    body: pullRequestMetadata.body.length > MAX_ALLOWED_BODY_CHARS ? "This repository has too many translation files, Mercury can't return a status description with the PR." : pullRequestMetadata.body
   };
 
   let handlePr;

src/resources/handle-pull-request.js

@@ -2,7 +2,7 @@
 
 const _ = require('lodash');
 const File = require('./github/file');
-const Octokit = require('@octokit/rest');
+const { Octokit } = require('@octokit/rest');
 const PullRequest = require('./github/pull-request');
 const RateLimit = require('./github/rate-limit');
 const Reference = require('./github/reference');
@@ -47,7 +47,10 @@ module.exports = config => {
       readOctokit.git
         .getTree(options)
         .then(({ data }) => {
-          next(null, _.map(data.tree, x => x.path));
+          next(
+            null,
+            _.map(data.tree, x => x.path)
+          );
         })
         .catch(err => next(err));
     });

src/services/github.js

@@ -1,7 +1,7 @@
 'use strict';
 
 const async = require('async');
-const Octokit = require('@octokit/rest');
+const { Octokit } = require('@octokit/rest');
 
 module.exports = config => ({
   get: next => {