Merge pull request #705 from primer/sync-refactors

Factor out Sync Functionality

Author: Emily

lib/add-bundle-meta.js

@@ -1,12 +1,9 @@
 /* eslint-disable no-console */
-const sync = require('./sync')
 const {NODE_ENV, NOW_URL} = process.env
 
 module.exports = (nextConfig = {}) => {
   const {assetPrefix = NOW_URL || ''} = nextConfig
 
-  let configured = false
-
   return Object.assign({}, nextConfig, {
     assetPrefix,
     pageExtensions: ['js', 'jsx', 'md', 'mdx'],
@@ -26,13 +23,6 @@ module.exports = (nextConfig = {}) => {
         )
       }
 
-      const {dev} = options
-
-      // only attempt to sync locally and in CI
-      if (dev && !configured) {
-        sync({watch: !NODE_ENV})
-      }
-
       config.module.rules.push({
         test: /\.svg$/,
         use: '@svgr/webpack'
@@ -43,7 +33,6 @@ module.exports = (nextConfig = {}) => {
         use: [options.defaultLoaders.babel, require.resolve('./mdx-loader')]
       })
 
-      configured = true
       if (typeof nextConfig.webpack === 'function') {
         return nextConfig.webpack(config, options)
       }

lib/add-source.js

@@ -39,12 +39,10 @@
     "start": "next dev",
     "dev": "next dev",
     "start-storybook": "start-storybook -p 8000 -c .storybook",
-    "sync": "script/sync",
     "test": "npm-run-all -s test-urls test-migrate",
     "test-all-modules": "ava --verbose tests/test-*.js",
-    "test-urls": "npm run sync && node docs-test/urls.js",
-    "test-migrate": "script/test-migrate",
-    "watch": "script/sync --watch"
+    "test-urls": "node docs-test/urls.js",
+    "test-migrate": "script/test-migrate"
   },
   "devDependencies": {
     "@githubprimer/octicons-react": "^8.1.3",