fix: add uncaughtException handler to deal with no config error

setup BRANCH env var in travis to fix this: if BRANCH=code then we fail
the script, if not we skip the script.

Author: zebateira

.travis.yml

@@ -1,3 +1,6 @@
+os:
+  - linux
+dist: xenial
 language: node_js
 node_js:
   - lts/*
@@ -26,11 +29,11 @@ notifications:
   email: false
 deploy:
   provider: pages
-  local-dir: dist
-  target-branch: master
-  skip-cleanup: true
-  github-token: $GITHUB_TOKEN  # Set in the settings page of your repository, as a secure variable
-  keep-history: true
+  local_dir: dist
+  target_branch: master
+  cleanup: true
+  token: $GITHUB_TOKEN  # Set in the settings page of your repository, as a secure variable
+  keep_history: true
   fqdn: proto.school
   on:
     branch: code

package-lock.json

@@ -64,7 +64,7 @@
   "dependencies": {
     "async": "^2.6.1",
     "cids": "^0.7.1",
-    "err-code": "^2.0.0",
+    "err-code": "^2.0.3",
     "googleapis": "^47.0.0",
     "highlight.js": "^9.12.0",
     "inquirer": "^7.1.0",

package.json

@@ -4,6 +4,22 @@
     - Google APIs for events data
  */
 
+let log = require('npmlog')
+
+const logGroup = 'build:data'
+
+process.on('uncaughtException', (error) => {
+  // We can't fail if the branch is code, because it could be the cron job
+  if (error.code === 'NO_CONFIG' && process.env.BRANCH !== 'code') {
+    log.error(logGroup, 'ERROR NO_CONFIG:', error.message)
+    log.warn(logGroup, 'Not enough config set. Skipping build-data script...')
+    process.exit(0)
+  }
+
+  log.error(logGroup, error.code ? `ERROR ${error.code}:` : '', error.message)
+  process.exit(1)
+})
+
 const run = require('../../modules/run')
 const eventsData = require('./events-data')
 const mailchimp = require('./mailchimp')

scripts/commands/build-data/index.js

@@ -2,11 +2,15 @@
     Create auth object for connecting with the Google APIs.
     All required information will be ready from env vars.
  */
+const errorCode = require('err-code')
 
 const { google } = require('googleapis')
 
 if (!process.env.GOOGLE_CLIENT_ID) {
-  throw new Error('No config available. Add a .env file or make all the config available through env variables. Please check the docs at https://github.com/ProtoSchool/protoschool.github.io/tree/code/scripts')
+  throw errorCode(
+    new Error('No config available. Add a .env file or make all the config available through env variables. Please check the docs at https://github.com/ProtoSchool/protoschool.github.io/tree/code/scripts'),
+    'NO_CONFIG'
+  )
 }
 
 const credentials = {