Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build appTheme.json using lockfile, allows concurrent build #335

Merged
merged 6 commits into from
Oct 5, 2016

Conversation

jordanh
Copy link
Contributor

@jordanh jordanh commented Oct 4, 2016

  • Uses a lockfile for building appTheme.json, concurrent builds and dependencies within webpack
  • Changed rethinkdb driver to return think, allows for delay of server pool

Try:

npm run build

Ooooh! Ahh!

"build": "rimraf build && npm run build:themes && concurrently \"npm run build:client\" \"npm run build:server\"",
"bs": "rimraf build && npm run build && npm start",
"build": "rimraf build && concurrently \"npm run build:client\" \"npm run build:server\"",
"bs": "npm run build && npm start",
"quickstart": "npm run db:migrate && npm run build && npm run start",
"build:client": "NODE_ENV=production webpack --config ./webpack/prod.babel.js",
"build:client-min": "NODE_ENV=production DEPLOY=true webpack --config ./webpack/prod.babel.js",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i noticed our staging server isn't set up to server the minimized version. Should we write a build:min script?

@jordanh jordanh merged commit 62c0552 into master Oct 5, 2016
@jordanh jordanh removed the pr review label Oct 5, 2016
@jordanh jordanh deleted the build-theme-with-lockfile branch October 5, 2016 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants