Merge pull request #238 from primer/path-alias

Make static assets available on path alias

.gitignore

@@ -1,6 +1,7 @@
 .*.swp
 .DS_Store
 *.log
+*.tgz
 .next/
 .test/
 coverage/

.travis.yml

@@ -7,10 +7,6 @@ cache:
     # cache jest intermediary files between builds
     - $(jq -r .jest.cacheDirectory package.json)
 
-# XXX remove this after merging
-before_script:
-  - npm i fs-extra
-
 script:
   - npm run lint
   # see <https://github.com/facebook/jest/issues/3855> for more info
@@ -19,6 +15,8 @@ script:
   - npm run dist
   # make sure we can build the site
   - npm run build
+  # make sure that the package includes and excludes all the right stuff
+  - npm pack
 
 notifications:
     slack: github:4OisIoqZYyMlWcw3hFtZgbmh

jest.config.js

@@ -0,0 +1,8 @@
+module.exports = {
+  cacheDirectory: '.test',
+  collectCoverage: true,
+  collectCoverageFrom: [
+    'src/*.js'
+  ],
+  setupTestFrameworkScriptFile: '<rootDir>/src/utils/test-matchers.js'
+}

next.config.js

@@ -1,7 +1,16 @@
-const withMDX = require('@zeit/next-mdx')({
-  extension: /\.(md|mdx)?$/
-})
+const withPlugins = require('next-compose-plugins')
+const sass = require('@zeit/next-sass')
+const mdx = require('@zeit/next-mdx')({extension: /\.mdx?$/})
 
-module.exports = withMDX({
-  pageExtensions: ['js', 'jsx', 'md', 'mdx']
+module.exports = withPlugins([sass, mdx], {
+  /*
+   * Note: Prefixing assets with the fully qualified deployment URL
+   * makes them available even when the site is served from a path alias, as in
+   * <https://primer.style/components>
+   */
+  assetPrefix: process.env.NOW_URL,
+  pageExtensions: ['js', 'jsx', 'md', 'mdx'],
+  sassLoaderOptions: {
+    includePaths: ['node_modules']
+  }
 })

now.json

@@ -1,9 +1,8 @@
 {
   "alias": "primer-react.now.sh",
   "files": [
+    "next.config.js",
     "pages",
-    "rollup.config.js",
-    "src",
-    "next.config.js"
+    "src"
   ]
 }