Chore: fix CI (#16)

* Prettify files

* Disable no-extraneous-dependencies eslint rule in the example, since there is no package.json

* Boilerplate: use "rudy-src-main" entry point in production mode (to avoid dependency on library builds)

* Correct value of "rudy-src-main" field in page title middleware and utils

* Add "build" script to all libraries so that they are built before the CI runs the integration tests

Author: hedgepigdaniel

.prettierignore

@@ -19,6 +19,7 @@ lerna-debug.log
 *.png
 *.snap
 *.log
+*.env
 
 # Ignored dirs in sub packages
 packages/*/

README.md

@@ -1,23 +1,34 @@
 # Respond Framework
-Think of your app in terms of _states_, not _routes_ or _components_. Connect your components and just dispatch _Flux Standard Actions_!
 
-**Respond Framework** is the successor to [redux-first-router](https://github.com/faceyspacey/redux-first-router).
+Think of your app in terms of _states_, not _routes_ or _components_. Connect
+your components and just dispatch _Flux Standard Actions_!
+
+**Respond Framework** is the successor to
+[redux-first-router](https://github.com/faceyspacey/redux-first-router).
 
 ## Motivation
-To be able to use Redux *as is* while keeping the address bar in sync. To achieve *bi-directional mapping* between the address bar and Redux actions. This way, changing the address (also via back/forward) dispatches an action, and dispatching an action changes the address. Paths are defined as actions, and path params and query strings are handled as action payloads.
 
-**Respond Framework** also seeks to *avoid* the following obstacles:
+To be able to use Redux _as is_ while keeping the address bar in sync. To
+achieve _bi-directional mapping_ between the address bar and Redux actions. This
+way, changing the address (also via back/forward) dispatches an action, and
+dispatching an action changes the address. Paths are defined as actions, and
+path params and query strings are handled as action payloads.
+
+**Respond Framework** also seeks to _avoid_ the following obstacles:
 
-* Rendering from state that doesn't come from Redux
-* Dealing with the added complexity from having state outside of Redux
-* Cluttering components with route-related code
-* Large API surface areas of frameworks like `react-router` and `next.js`
-* Routing frameworks getting in the way of optimizing animations (such as when animations coincide with component updates)
-* Having to do route changes differently in order to support server-side rendering
+- Rendering from state that doesn't come from Redux
+- Dealing with the added complexity from having state outside of Redux
+- Cluttering components with route-related code
+- Large API surface areas of frameworks like `react-router` and `next.js`
+- Routing frameworks getting in the way of optimizing animations (such as when
+  animations coincide with component updates)
+- Having to do route changes differently in order to support server-side
+  rendering
 
 ## Usage
 
 ### Install
+
 `yarn add @respond-framework/rudy`
 
 ### Basic example for React
@@ -165,7 +176,9 @@ export {
 
 The source code for this example can be found [here](./examples/react).
 
-Also see the [boilerplate project](./packages/boilerplate) and the (partly outdated) [documentation](./packages/rudy/docs).
+Also see the [boilerplate project](./packages/boilerplate) and the (partly
+outdated) [documentation](./packages/rudy/docs).
 
 ## License
+
 [MIT](LICENSE)

examples/react/basic/.eslintrc.js

@@ -3,13 +3,14 @@ const path = require('path')
 module.exports = {
   extends: [path.resolve(__dirname, '../../../.eslintrc.js')],
   env: {
-    browser: true
+    browser: true,
   },
   rules: {
     'no-param-reassign': 0,
     'import/no-unresolved': 0,
     'import/extensions': 0,
     'react/jsx-filename-extension': 0,
-    'react/prop-types': ['error', { skipUndeclared: true }]
+    'react/prop-types': ['error', { skipUndeclared: true }],
+    'import/no-extraneous-dependencies': 0, // There is no package.json
   },
 }

packages/boilerplate/server/webpack.config.babel.js

@@ -90,7 +90,7 @@ export default (env) => {
         ? ['.server.js', '.js', '.css']
         : ['.browser.js', '.js', '.css'],
       mainFields: [
-        isDev && 'rudy-src-main',
+        'rudy-src-main',
         isClient && 'browser',
         'module',
         'main',

packages/link/package.json

@@ -20,6 +20,7 @@
     "prepare": "yarn run build:cjs && yarn run build:es && yarn run flow-copy",
     "build:cjs": "babel --root-mode upward --source-maps true src -d cjs",
     "build:es": "BABEL_ENV=es babel --root-mode upward --source-maps true src -d es",
+    "build": "yarn run build:cjs && yarn run build:es",
     "clean": "rimraf cjs && rimraf es",
     "flow-copy": "flow-copy-source src cjs && flow-copy-source src es",
     "prettier": "prettier",

packages/middleware-change-page-title/package.json

@@ -4,7 +4,7 @@
   "description": "Rudy middleware to synchronise the browser page title with redux state",
   "main": "cjs/index.js",
   "module": "es/index.js",
-  "rudy-src-main": "src/index.jsx",
+  "rudy-src-main": "src/index.js",
   "repository": "https://github.com/respond-framework/rudy.git",
   "contributors": [
     "James Gilmore <[email protected]>",
@@ -23,6 +23,7 @@
     "prepare": "yarn run build:cjs && yarn run build:es && yarn run flow-copy",
     "build:cjs": "babel --root-mode upward --source-maps true src -d cjs",
     "build:es": "BABEL_ENV=es babel --root-mode upward --source-maps true src -d es",
+    "build": "yarn run build:cjs && yarn run build:es",
     "clean": "rimraf cjs && rimraf es",
     "flow-copy": "flow-copy-source src cjs && flow-copy-source src es",
     "prettier": "prettier",

packages/rudy/docs/differences-from-rfr.md

@@ -1,6 +1,9 @@
 # Differences from [redux-first-router](https://github.com/faceyspacey/redux-first-router)
 
-* `[NOT_FOUND]` -> `"NOT_FOUND"` in reducers (e.g. page reducer).
-* `connectRoutes` has been renamed to `createRouter` and does not return `enhancer` anymore.
-* `firstRoute`, now returned from `createRouter`, must be invoked before rendering.
-* `payload` has been renamed to `params`, e.g. `{ type: 'FOO', params: {} }` and `state.location.params`.
+- `[NOT_FOUND]` -> `"NOT_FOUND"` in reducers (e.g. page reducer).
+- `connectRoutes` has been renamed to `createRouter` and does not return
+  `enhancer` anymore.
+- `firstRoute`, now returned from `createRouter`, must be invoked before
+  rendering.
+- `payload` has been renamed to `params`, e.g. `{ type: 'FOO', params: {} }` and
+  `state.location.params`.

packages/rudy/package.json

@@ -15,6 +15,7 @@
     "build:es": "BABEL_ENV=es babel --root-mode upward --source-maps true src -d es",
     "build:umd": "NODE_ENV=production webpack --mode production --env.minimize=false src/index.js -o dist/redux-first-router.js",
     "build:umd:min": "NODE_ENV=production webpack --mode production --env.minimize=true src/index.js -o dist/redux-first-router.min.js",
+    "build": "yarn run build:cjs && yarn run build:es && yarn run build:umd && yarn run build:umd:min",
     "flow-copy": "flow-copy-source src dist && flow-copy-source src es",
     "flow-watch": "clear; printf \"\\033[3J\" & npm run flow & fswatch -o ./ | xargs -n1 -I{} sh -c 'clear; printf \"\\033[3J\" && npm run flow'",
     "flow": "flow; test $? -eq 0 -o $? -eq 2",

packages/utils/package.json

@@ -4,7 +4,7 @@
   "description": "Shared utilities for rudy",
   "main": "cjs/index.js",
   "module": "es/index.js",
-  "rudy-src-main": "src/index.jsx",
+  "rudy-src-main": "src/index.js",
   "repository": "https://github.com/respond-framework/rudy.git",
   "contributors": [
     "James Gilmore <[email protected]>",
@@ -23,6 +23,7 @@
     "prepare": "yarn run build:cjs && yarn run build:es && yarn run flow-copy",
     "build:cjs": "babel --root-mode upward --source-maps true src -d cjs",
     "build:es": "BABEL_ENV=es babel --root-mode upward --source-maps true src -d es",
+    "build": "yarn run build:cjs && yarn run build:es",
     "clean": "rimraf cjs && rimraf es",
     "flow-copy": "flow-copy-source src cjs && flow-copy-source src es",
     "prettier": "prettier",