[WIP] Get the before and after for the build results

Author: orta

dangerfile.js

@@ -9,15 +9,35 @@
 
 'use strict';
 
-const { danger, markdown, warn } = require('danger');
+const { danger, markdown, schedule } = require('danger');
 
-// Tags big PRs
-var bigPRThreshold = 600;
-if (danger.git.modified_files + danger.git.added_files + danger.git.deleted_files > bigPRThreshold) {
-  const title = ':exclamation: Big PR';
-  const files = danger.git.modified_files + danger.git.added_files + danger.git.deleted_files;
-  const idea = `This PR is extremely unlikely to get reviewed because it touches ${files} files.`;
-  warn(`${title} - <i>${idea}</i>`);
+const { resultsHeaders, generateResultArray, currentBuildResults } = require('./scripts/rollup/stats');
 
-  markdown('@facebook-github-bot large-pr');  
+/**
+ * Generates a Markdown table
+ * @param {string[]} headers 
+ * @param {string[][]} body 
+ */
+function generateMDTable(headers, body) {
+  const tableHeaders = [
+    headers.join(' | '), 
+    headers.map(() => ' --- ').join(' | '),
+  ];
+
+  const tablebody = body.map(r => r.join(' | '));
+  return tableHeaders.join('\n') + '\n' + tablebody.join('\n');
 }
+
+// Grab the results.json before we ran CI via the GH API
+const getJSON = danger.github.utils
+  .fileContents('scripts/rollup/results.json');
+
+// @bug See https://github.com/danger/danger-js/issues/443
+schedule(getJSON);
+getJSON.then(APIPreviousBuildResults => {
+  const previousBuildResults = JSON.parse(APIPreviousBuildResults);
+  const results = generateResultArray(currentBuildResults, previousBuildResults);
+
+  markdown('### Bundle Changes:\n');
+  markdown(generateMDTable(resultsHeaders, results));
+});

scripts/circleci/test_entry_point.sh

@@ -25,11 +25,11 @@ if [ $((2 % CIRCLE_NODE_TOTAL)) -eq "$CIRCLE_NODE_INDEX" ]; then
   COMMANDS_TO_RUN+=('./scripts/circleci/build.sh')
   COMMANDS_TO_RUN+=('yarn test-build --runInBand')
   COMMANDS_TO_RUN+=('yarn test-build-prod --runInBand')
+  COMMANDS_TO_RUN+=('node ./scripts/tasks/danger')
   COMMANDS_TO_RUN+=('./scripts/circleci/upload_build.sh')
 fi
 
 if [ $((3 % CIRCLE_NODE_TOTAL)) -eq "$CIRCLE_NODE_INDEX" ]; then
-  COMMANDS_TO_RUN+=('node ./scripts/tasks/danger')
   COMMANDS_TO_RUN+=('./scripts/circleci/test_coverage.sh')
 fi
 

scripts/rollup/stats.js

@@ -70,4 +70,5 @@ module.exports = {
   printResults,
   saveResults,
   currentBuildResults,
+  prevBuildResults,
 };

scripts/tasks/danger.js

@@ -7,18 +7,18 @@
 
 'use strict';
 
-var path = require('path');
-var spawn = require('child_process').spawn;
+const path = require('path');
+const spawn = require('child_process').spawn;
 
-var extension = process.platform === 'win32' ? '.cmd' : '';
+const extension = process.platform === 'win32' ? '.cmd' : '';
 
-// This came from React Native's circle.yaml
+// This came from React Native's circle.yml
 const token = 'e622517d9f1136ea8900' + '07c6373666312cdfaa69';
 spawn(path.join('node_modules', '.bin', 'danger' + extension), [], {
   // Allow colors to pass through
   stdio: 'inherit',
   env: {
-    ...process.env, 
+    ...process.env,
     DANGER_GITHUB_API_TOKEN: token,
   },
 }).on('close', function(code) {