/react-boilerplate

Move test #1432

Merged
merged 9 commits into from Jan 13, 2017
+19 −10

Conversation

Reviewers

@Dattaya Dattaya

@mxstbr mxstbr

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@gihrig @Dattaya @mxstbr
@gihrig
Owner

gihrig commented Jan 10, 2017

Now that we are focussing more on tests and requiring a high percentage of test coverage for CI builds to pass, there is more reason than ever to allow building when tests are incomplete. This is a pain for manual acceptance testing.

This PR moves toward a better single responsibility for build and test scripts. If you want to manually choose processes, select from test, build and start:prod If you know you want it all start:production.

This PR:

  1. Removes the testing phase from build.
  2. Adds a testing phase to `start:production.
  3. Modifies .travis.yaml and appveyor.yaml to run tests in the build process after the above.
  4. Clarifies appropriate docs.
  5. Removes a minor (manual) acceptance testing annoyance from selectors.test.js generator.

gihrig added some commits Dec 31, 2016

@gihrig gihrig feature(test): enable build without test 
Git pre-commit hook still prevents submit failing code
 1325881
@gihrig gihrig docs(test): updated documentation 
Clarified test, build and start:production
 66617be
@gihrig gihrig test(selector test generator): change 'test case' to 'true' 
This make it easier 'cheat' on the test (double-click to select). Useful when manually testing generator output.
 72a1c77
@gihrig gihrig docs(test): updated documentation 
Clarified start:production

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
a22a013
internals/generators/container/selectors.test.js.hbs
@@ -5,6 +5,6 @@
describe('makeSelect{{ properCase name }}Domain', () => {
it('Expect to have unit tests specified', () => {
- expect('Test case').toEqual(false);
+ expect('true).toEqual(false);
@Dattaya

Dattaya Jan 10, 2017

Member

Found a redundant single quote in front of true, other than that, I support this PR.

gihrig added some commits Jan 10, 2017

@gihrig gihrig remove erroneous single quote

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
e12eca1
@gihrig gihrig Merge branch 'dev' into move-test

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
d8ae043
@gihrig
Owner

gihrig commented Jan 10, 2017

Thanks @Dattaya ! Fixed that ✨
@Dattaya
Member

Dattaya commented Jan 10, 2017

Awesome 👍
@gihrig
Owner

gihrig commented Jan 10, 2017

@Collaborators Can we get another approval and land this? It puts a finishing touch on @outdooricon's excellent work on tests in #1414. Which has been merged and will be included in 3.4.

Thanks ✨

Dattaya requested a review from mxstbr Jan 10, 2017

gihrig added some commits Jan 12, 2017

@gihrig gihrig Merge branch 'dev' into move-test

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
9ce1721
@gihrig gihrig Merge branch 'dev' into move-test

Some checks were not successful

1 errored and 1 successful checks
continuous-integration/travis-ci/pr — The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
bc10bae
@gihrig gihrig test(travis): revert yarn calls to npm 
Travis does not directly support yarn

As described at https://blog.travis-ci.com/2016-11-21-travis-ci-now-supports-yarn
Travis looks for yarn.lock and maps npm calls to yarn, if present.

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
826f344

gihrig merged commit 4155217 into react-boilerplate:dev Jan 13, 2017
2 checks passed

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@Dattaya
Member

Dattaya commented Jan 13, 2017

Travis does not directly support yarn
As described at https://blog.travis-ci.com/2016-11-21-travis-ci-now-supports-yarn
Travis looks for yarn.lock and maps npm calls to yarn, if present.

well, this line from the blog: If your Node.js project has the yarn.lock file in the repository root, we proceed to install Yarn and execute the yarn command instead of the default npm install. in my understanding means that Travis would automatically call yarn install if .yarn.lock is present no matter what's inside .travis.yml file, so that means no npm calls in the config file are going to be mapped to yarn.

We probably also need to correct a bit .travis.yml so we test in the scripts section:

script:
  - node ./internals/scripts/generate-templates-for-linting
  - npm run test
  - npm run build
@mxstbr
Owner

mxstbr commented Jan 13, 2017

Sorry for the late response, love this.

Dattaya referenced this pull request Jan 13, 2017

Merged

Introduction cleanup #1449

@Dattaya Dattaya added a commit to Dattaya/react-boilerplate that referenced this pull request Jan 13, 2017

@Dattaya Dattaya chore(ci): fix small regression in #1432 0d76880

This was referenced Jan 13, 2017

Closed
Merged
@gihrig
Owner

gihrig commented Jan 13, 2017

@Dattaya Thanks for looking into this 😄

that means no npm calls in the config file are going to be mapped to yarn.

  • Not sure what to do with this... Replace all npm calls with yarn calls? I did that at first and it failed on my fork/Travis. So restored npm calls and it passed.

We probably also need to correct a bit .travis.yml so we test in the scripts section:

  • Not sure why those were in install section. Moved to scripts.

  • Travis builds succeed, now as before using yarn (on my fork). Not sure any of this has to do with travis.yaml...
    image

Will open a new PR with move from install -> scripts. See #1455

gihrig referenced this pull request Jan 13, 2017

Closed

chore(ci): fix small regression in #1432 #1453

2 of 7 tasks complete
@Dattaya
Member

Dattaya commented Jan 13, 2017

Replace all npm calls with yarn calls?

For some reason I thought you wanted to restore npm completely, just running scripts with npm instead of yarn, I don't think it matters. So everything's fine.

@AnhHT AnhHT added a commit to AnhHT/react-boilerplate that referenced this pull request Jan 18, 2017

@AnhHT AnhHT Merge with base (#2) 
* feat(docs): Extracting packages into their own modules (#979)

* fix(internals): Fix babel relative preset

Fix webpack issue with relative babel preset paths:
babel/babel-loader#149

* feat(docs): Add docs for extracting components

Add draft version of docs describing how to extract components to their own
npm packages.

* feat(core): Move to Jest for testing (#980)

* Add Jest

collect coverage

* fix(tests): set config and globals

add coverage and script

update coveralls script

update chai to jest syntax

args should return an array

chore(tests): add babel-jest, tweak config

chore(tests): update coverage settings

feat(cov): remove untested closure

chore(core): update watch script, use jest-cli

fix(watch): add no-cache and watchAll flags

* Remove generator test

* jest-cli@16

* chore(jest): Remove rootDir

* rebase against 3.3.0

* fix(commands): Fix 'npm run clean' breaking (#1185) (#1186)

Added a constants.js file to templates that contains the DEFAULT_LOCALE
export to be added to containers/App/constants on 'npm run clean'.

* fix(example): Fix i18n button not updating (#1187)

* feat(docs): Faq: staying updated with boilerplate (#1192)

* fix(example): Fix console error on lang change (#1200)

* fix(core): npm run generate language is not properly created. (#1199)

* rerun extract-intl

* add default language to language provider

* feat(webpack): new query object syntax (#1198)

* feat(ci): Node v7 support (#1197)

* Node version support update

* Update CI node versions

* fix(docs): Replace start:prod with independent start:production (#1204)

* fix(example): Fix default Intl polyfill language (#1208)

* fix(docs): Fix typo in file name (#1211)

* fix(example): Use `createStructuredSelector` instead of `createSelector` (#1212)

* chore(deps): update deps roll-up Nov-12-2016 (#1225)

* chore(deps): update deps roll-up Nov-19-2016 (#1252)

* Prevent language duplication (#1253)

* Satisfy ESLint rule: no-restricted-syntax (#1243)

* feat(webpack) Add circular dependency plugin for Webpack. (#1262)

* feat(webpack) Add circular dependency plugin for Webpack.

This plugin detects circular dependencies and shows a warning or fails.
It is less obtrusive to just show a warning. That's why it defaults to
that.

* small fix to webpack config

* Remove webpack DedupePlugin (#1218)

* Fix typo in container generator (#1242)

* fix(eslint config): resolve numerous invalid lint errors seen in Atom editor (#1261)

* Disable ESLint rule: no-confusing-arrow (#1238)

* Fix links to styled-components doc (#1289)

* Add missing document (Using Sass section CSS) to table content (#1271)

* fix #1270

* add brackets to document Using Sass

* Remove '#if wantCSS' from generator templates (#1260)

Remove '#if wantCSS' from container generator

Remove '#if wantCSS' from es6 pure generator template

* feat(core): Don't import all of lodash

* fix(package.json): Required npm version (#1219)

* Fix wrong npm version introduced with d202b07

Commit d202b07 introduced a not existing npm version of >= 5 which does not exist. The correct npm version is checked during installation which might otherwise lead to an error. Conclusion might be that this was meant to be node >= 5 like mentioned in #1197

* Add the supported version of node

* chore(deps): update deps roll-up Dec-03-2016 (#1301)

* chore(deps): update deps roll-up Dec-03-2016

* fix(deps): sagas loaded twice - roll back react-router-redux 4.0.7 -> 4.0.6

* chore(test config): expand Jest moduleNameMapper key to include common file types (#1293)

* chore(selectors): refactor selectors (#1205)

* Improve .editorconfig, change insert_final_newline to true, add trim_trailing_whitespace (#1239)

* fix(scripts): make LocaleToggle's massages compatible with `extract-intl` script (#1251)

Fixes #1221
Reverts back #813

* feat(example): Extracted ReposList (#1285)

* Bfix/reorder imports (#1275)

* Reorder imports

* Track package.json

* Reorder remaining imports

* Revert few changes in app/app.js

* Add .ico extension to file-loader regex

* Revert favicon, manifest, .htaccess imports

* Revert favicon, manifest, .htaccess imports

* Remove ico from image-webpack loader extensions

* Move offline-plugin import to the top in templates/app.js

* Match templates/app.js with app/app.js

* Remove import/first from package.json

* Reorder webpack alias imports

* Reorder imports in internals/generators

* Remove German from templates/app.js

* fix(templates): use correct name of a selector (#1321)

* chore(generators): clean up generator templates (#1304)

Follow up to #1260

* fix(generator):  #1231 'generate container' produces unnecessary return (#1327)

* chore(internal): fix indentation in 'routeWithReducer' (#1329)

* fix(HomePage): #1232 Saga effects should be yielded (#1328)

* fix(generator): 'generate container' produces unnecessary return - better solution (#1333)

* fix(generator):  #1231 'generate container' produces unnecessary return

* fix(generator): improved solution to 'generate container' produces unnecessary return #1327

* feat(ci): allow for linting of the templates (#1223)

* feat(ci): allow for linting of the templates

*  Add comment at the top describing the purpose of generate-templates-for-linting.js

* Use __dirname instead of process.cwd()

* Update yarn.lock

* Test coverage updates (#1334)

* Exclude test files from test coverage

* Add coverage threshold

* Update css readme (#1338)

* fix(generators, templates): add example app functionality into project templates and generators (#1331)

* fix(generators, templates): add existing example app functionality into project templates and generators

* fix(generators, templates): add existing example app functionality into project templates and generators

* split messages import to avoid disabling an eslint rule

* feat(server): Add host command line parameter to start (#1249)

* add host parameter to logger

* add host parameter to server

* update docs to include host command line parameter

* update host port command doc based on suggesitons

* Rollback image-webpack-loader (#1342)

* Remove babel-jest expicit dependency (#1345)

implements #1344

* Make  compatible with other babel plugins (#1355)

* Remove psi and pagespeed (#1352)

* removing psi, pagespeed, and doc references

* remove psi from yarn.lock

* Trigger full page reload when hmr fails (#1357)

Fixes #1353

* refactor(server): remove console.log causing server screen clutter (#1361)

* feat(example): Add route loading progress bar (#1008)

* Add ProgressBar, modify App and write tests

* fix eslint warnings

* Prevent memory leak in App/index, Fix animation of ProgressBar, Add more tests

* Add more tests for ProgressBar life cycle methods

* Rename css class names, remove eslint-disable, add comments

* Remove comments from ProgressBar tests

* Prevent re-rendering of ProgressBar

* Fix failing test

* Add one more condition to complete progress

* Update index.test.js

Bring import order in line with #1275

* Update App/index.js

* Update App/index.js

* Fix re-rendering and make tests pass

* Move progress route hooks out of <App />, update tests

* Reset progress bar speed

* Move <ProgressBar /> out of <App />

* Update tests

* Move withProgressBar to components/ProgressBar

* Rename withProgressBar -> index.js.

* Add docs for ProgressBar working/usage

* Update progress bar.md

* Split ProgressBar/styles.js into two separate files

* chore(package.json): update dependecy

The test/build failed due to a regressed dependency. 

"react-addons-test-utils": "15.3.2", ->  "react-addons-test-utils": "15.4.1",

* feat(docs) a beginner's guide to RBP (#1366)

* Create introduction.md

* Update introduction.md

Minor edits for grammar and structure.

Overall, really really good. The hours of dedicated effort you put in will be appreciated by many to come! Very well done 😄

* Update introduction.md

Here I have implemented the changes mentioned by @web2style and @lamoglia Thanks guys! :spakles:

* Host workflow.png internally

* Reference local image in markdown

* Rename Workflow.png -> workflow.png

* Move to Jest Documentation (#1390)

* Move to Jest Documentation

- Move from Mocha to Jest documentation
- Remove rewire docs and replace with Jest snapshot information

* Minor edits

Overall very good, only a few minor edits for style.

* Remove jest-cli cmd

* Update commands.md

* Update deployment.md

* fix(core): ngrok not working (#1399)

Node's http.Server#listen does some smart choosing for default host '::' or '0.0.0.0' depending on IPv6 support. Use that instead of binding to localhost specifically. This mimics previous server functionality and allows ngrok access to work again.

* chore(deps): updated redux-saga version (#1388)

* Update Changelog.md

* Update package.json

* 3.4.0

* 3.4.0

* 3.4.0

* docs(toc): include link to new intor document (#1401)

* Update Changelog

* chore(deps): update webpack (#1358)

* chore(deps): update webpack

* Update 'Introduction'

* Cache again

* Fix warning during generation of webpack dll file

* Describe how to turn off the warnings in production

* Bump webpack

* Improve the docs

* Reorg some templates and add tests (#1414)

* Reorg some templates and add tests

- Also make sure that we keep LanguageProvider tests after clean

* Move to container structure for internals

* Remove old filename ref

* Actually remove folders, thanks @dattaya

* Fix #1300 broken link (#1429)

* Fix #1435 comment (#1437)

* Add AWS documentation to deployment.md (#1442)

* Add AWS documentation to deployment.md

* Minor changes

* feature(nginx): integrating mozillas nginx secure tls configuration i… (#1393)

* feature(nginx): integrating mozillas nginx secure tls configuration into .nginx.conf

* feature(nginx): updating server-config docs for apache and nginx to reflect security configurations

* Update .nginx.conf

Add Link for DHE Handshake and minor edits for style/grammar

* Update server-configs.md

Minor edits for style/grammar

* RFC: chore(ci): switch to yarn (#1415)

* chore(ci): switch to yarn

* Remove 'yarn version' because: 'Can't answer a question unless a user TTY'

* chore(deps): remove chai and chai-enzyme (#1441)

* #1406 remove lint:css documentation references (#1445)

* Removed redundant lint:css

Remove lint:css references

* Removed redundant lint:js reference

* docs(ISSUE_TEMPLATE): add request for RBP version (#1425)

As there are often several version in various state of completion and we attempt to release often. It's vital to know what version of RBP is involved in issues.

* chore(internals): do not check file sizes of main chunk and favicon (#1424)

* chore(internals): do not check file sizes of main chunk and favicon

* Reword explanation in the docs

* Move test (#1432)

* feature(test): enable build without test

Git pre-commit hook still prevents submit failing code

* docs(test): updated documentation

Clarified test, build and start:production

* test(selector test generator): change 'test case' to 'true'

This make it easier 'cheat' on the test (double-click to select). Useful when manually testing generator output.

* docs(test): updated documentation

Clarified start:production

* remove erroneous single quote

* test(travis): revert yarn calls to npm

Travis does not directly support yarn

As described at https://blog.travis-ci.com/2016-11-21-travis-ci-now-supports-yarn
Travis looks for yarn.lock and maps npm calls to yarn, if present.

* config(travis.yaml): move test and build to script section (#1455)

* chore(docs): cleanup of introduction

* chore(docs): cleanup of introduction

* chore(readme): add hitchhikers guide to the readme

* Several minor edits...

* Remove (in no particular order)

* docs(servers): update server conf files for offline-first per #645 (#1381)

* docs(servers): update server conf files for offline-first per #645

re caching see mxstbr#645 (comment)

Apache
http://stackoverflow.com/a/33287352/522756

Nginx
https://www.nginx.com/blog/nginx-caching-guide/

* Update .gitattributes

added "." to nginx.conf

* Remove empty line in list

* Picked a nit :-)

* chore(core): Move mocks/ folder to internals/ (#1460)

* Update CHANGELOG
 034ab9d

@AnhHT AnhHT added a commit to AnhHT/react-boilerplate that referenced this pull request Jan 20, 2017

@AnhHT AnhHT Merge with base (#3) 
* feat(docs): Extracting packages into their own modules (#979)

* fix(internals): Fix babel relative preset

Fix webpack issue with relative babel preset paths:
babel/babel-loader#149

* feat(docs): Add docs for extracting components

Add draft version of docs describing how to extract components to their own
npm packages.

* feat(core): Move to Jest for testing (#980)

* Add Jest

collect coverage

* fix(tests): set config and globals

add coverage and script

update coveralls script

update chai to jest syntax

args should return an array

chore(tests): add babel-jest, tweak config

chore(tests): update coverage settings

feat(cov): remove untested closure

chore(core): update watch script, use jest-cli

fix(watch): add no-cache and watchAll flags

* Remove generator test

* jest-cli@16

* chore(jest): Remove rootDir

* rebase against 3.3.0

* fix(commands): Fix 'npm run clean' breaking (#1185) (#1186)

Added a constants.js file to templates that contains the DEFAULT_LOCALE
export to be added to containers/App/constants on 'npm run clean'.

* fix(example): Fix i18n button not updating (#1187)

* feat(docs): Faq: staying updated with boilerplate (#1192)

* fix(example): Fix console error on lang change (#1200)

* fix(core): npm run generate language is not properly created. (#1199)

* rerun extract-intl

* add default language to language provider

* feat(webpack): new query object syntax (#1198)

* feat(ci): Node v7 support (#1197)

* Node version support update

* Update CI node versions

* fix(docs): Replace start:prod with independent start:production (#1204)

* fix(example): Fix default Intl polyfill language (#1208)

* fix(docs): Fix typo in file name (#1211)

* fix(example): Use `createStructuredSelector` instead of `createSelector` (#1212)

* chore(deps): update deps roll-up Nov-12-2016 (#1225)

* chore(deps): update deps roll-up Nov-19-2016 (#1252)

* Prevent language duplication (#1253)

* Satisfy ESLint rule: no-restricted-syntax (#1243)

* feat(webpack) Add circular dependency plugin for Webpack. (#1262)

* feat(webpack) Add circular dependency plugin for Webpack.

This plugin detects circular dependencies and shows a warning or fails.
It is less obtrusive to just show a warning. That's why it defaults to
that.

* small fix to webpack config

* Remove webpack DedupePlugin (#1218)

* Fix typo in container generator (#1242)

* fix(eslint config): resolve numerous invalid lint errors seen in Atom editor (#1261)

* Disable ESLint rule: no-confusing-arrow (#1238)

* Fix links to styled-components doc (#1289)

* Add missing document (Using Sass section CSS) to table content (#1271)

* fix #1270

* add brackets to document Using Sass

* Remove '#if wantCSS' from generator templates (#1260)

Remove '#if wantCSS' from container generator

Remove '#if wantCSS' from es6 pure generator template

* feat(core): Don't import all of lodash

* fix(package.json): Required npm version (#1219)

* Fix wrong npm version introduced with d202b07

Commit d202b07 introduced a not existing npm version of >= 5 which does not exist. The correct npm version is checked during installation which might otherwise lead to an error. Conclusion might be that this was meant to be node >= 5 like mentioned in #1197

* Add the supported version of node

* chore(deps): update deps roll-up Dec-03-2016 (#1301)

* chore(deps): update deps roll-up Dec-03-2016

* fix(deps): sagas loaded twice - roll back react-router-redux 4.0.7 -> 4.0.6

* chore(test config): expand Jest moduleNameMapper key to include common file types (#1293)

* chore(selectors): refactor selectors (#1205)

* Improve .editorconfig, change insert_final_newline to true, add trim_trailing_whitespace (#1239)

* fix(scripts): make LocaleToggle's massages compatible with `extract-intl` script (#1251)

Fixes #1221
Reverts back #813

* feat(example): Extracted ReposList (#1285)

* Bfix/reorder imports (#1275)

* Reorder imports

* Track package.json

* Reorder remaining imports

* Revert few changes in app/app.js

* Add .ico extension to file-loader regex

* Revert favicon, manifest, .htaccess imports

* Revert favicon, manifest, .htaccess imports

* Remove ico from image-webpack loader extensions

* Move offline-plugin import to the top in templates/app.js

* Match templates/app.js with app/app.js

* Remove import/first from package.json

* Reorder webpack alias imports

* Reorder imports in internals/generators

* Remove German from templates/app.js

* fix(templates): use correct name of a selector (#1321)

* chore(generators): clean up generator templates (#1304)

Follow up to #1260

* fix(generator):  #1231 'generate container' produces unnecessary return (#1327)

* chore(internal): fix indentation in 'routeWithReducer' (#1329)

* fix(HomePage): #1232 Saga effects should be yielded (#1328)

* fix(generator): 'generate container' produces unnecessary return - better solution (#1333)

* fix(generator):  #1231 'generate container' produces unnecessary return

* fix(generator): improved solution to 'generate container' produces unnecessary return #1327

* feat(ci): allow for linting of the templates (#1223)

* feat(ci): allow for linting of the templates

*  Add comment at the top describing the purpose of generate-templates-for-linting.js

* Use __dirname instead of process.cwd()

* Update yarn.lock

* Test coverage updates (#1334)

* Exclude test files from test coverage

* Add coverage threshold

* Update css readme (#1338)

* fix(generators, templates): add example app functionality into project templates and generators (#1331)

* fix(generators, templates): add existing example app functionality into project templates and generators

* fix(generators, templates): add existing example app functionality into project templates and generators

* split messages import to avoid disabling an eslint rule

* feat(server): Add host command line parameter to start (#1249)

* add host parameter to logger

* add host parameter to server

* update docs to include host command line parameter

* update host port command doc based on suggesitons

* Rollback image-webpack-loader (#1342)

* Remove babel-jest expicit dependency (#1345)

implements #1344

* Make  compatible with other babel plugins (#1355)

* Remove psi and pagespeed (#1352)

* removing psi, pagespeed, and doc references

* remove psi from yarn.lock

* Trigger full page reload when hmr fails (#1357)

Fixes #1353

* refactor(server): remove console.log causing server screen clutter (#1361)

* feat(example): Add route loading progress bar (#1008)

* Add ProgressBar, modify App and write tests

* fix eslint warnings

* Prevent memory leak in App/index, Fix animation of ProgressBar, Add more tests

* Add more tests for ProgressBar life cycle methods

* Rename css class names, remove eslint-disable, add comments

* Remove comments from ProgressBar tests

* Prevent re-rendering of ProgressBar

* Fix failing test

* Add one more condition to complete progress

* Update index.test.js

Bring import order in line with #1275

* Update App/index.js

* Update App/index.js

* Fix re-rendering and make tests pass

* Move progress route hooks out of <App />, update tests

* Reset progress bar speed

* Move <ProgressBar /> out of <App />

* Update tests

* Move withProgressBar to components/ProgressBar

* Rename withProgressBar -> index.js.

* Add docs for ProgressBar working/usage

* Update progress bar.md

* Split ProgressBar/styles.js into two separate files

* chore(package.json): update dependecy

The test/build failed due to a regressed dependency. 

"react-addons-test-utils": "15.3.2", ->  "react-addons-test-utils": "15.4.1",

* feat(docs) a beginner's guide to RBP (#1366)

* Create introduction.md

* Update introduction.md

Minor edits for grammar and structure.

Overall, really really good. The hours of dedicated effort you put in will be appreciated by many to come! Very well done 😄

* Update introduction.md

Here I have implemented the changes mentioned by @web2style and @lamoglia Thanks guys! :spakles:

* Host workflow.png internally

* Reference local image in markdown

* Rename Workflow.png -> workflow.png

* Move to Jest Documentation (#1390)

* Move to Jest Documentation

- Move from Mocha to Jest documentation
- Remove rewire docs and replace with Jest snapshot information

* Minor edits

Overall very good, only a few minor edits for style.

* Remove jest-cli cmd

* Update commands.md

* Update deployment.md

* fix(core): ngrok not working (#1399)

Node's http.Server#listen does some smart choosing for default host '::' or '0.0.0.0' depending on IPv6 support. Use that instead of binding to localhost specifically. This mimics previous server functionality and allows ngrok access to work again.

* chore(deps): updated redux-saga version (#1388)

* Update Changelog.md

* Update package.json

* 3.4.0

* 3.4.0

* 3.4.0

* docs(toc): include link to new intor document (#1401)

* Update Changelog

* chore(deps): update webpack (#1358)

* chore(deps): update webpack

* Update 'Introduction'

* Cache again

* Fix warning during generation of webpack dll file

* Describe how to turn off the warnings in production

* Bump webpack

* Improve the docs

* Reorg some templates and add tests (#1414)

* Reorg some templates and add tests

- Also make sure that we keep LanguageProvider tests after clean

* Move to container structure for internals

* Remove old filename ref

* Actually remove folders, thanks @dattaya

* Fix #1300 broken link (#1429)

* Fix #1435 comment (#1437)

* Add AWS documentation to deployment.md (#1442)

* Add AWS documentation to deployment.md

* Minor changes

* feature(nginx): integrating mozillas nginx secure tls configuration i… (#1393)

* feature(nginx): integrating mozillas nginx secure tls configuration into .nginx.conf

* feature(nginx): updating server-config docs for apache and nginx to reflect security configurations

* Update .nginx.conf

Add Link for DHE Handshake and minor edits for style/grammar

* Update server-configs.md

Minor edits for style/grammar

* RFC: chore(ci): switch to yarn (#1415)

* chore(ci): switch to yarn

* Remove 'yarn version' because: 'Can't answer a question unless a user TTY'

* chore(deps): remove chai and chai-enzyme (#1441)

* #1406 remove lint:css documentation references (#1445)

* Removed redundant lint:css

Remove lint:css references

* Removed redundant lint:js reference

* docs(ISSUE_TEMPLATE): add request for RBP version (#1425)

As there are often several version in various state of completion and we attempt to release often. It's vital to know what version of RBP is involved in issues.

* chore(internals): do not check file sizes of main chunk and favicon (#1424)

* chore(internals): do not check file sizes of main chunk and favicon

* Reword explanation in the docs

* Move test (#1432)

* feature(test): enable build without test

Git pre-commit hook still prevents submit failing code

* docs(test): updated documentation

Clarified test, build and start:production

* test(selector test generator): change 'test case' to 'true'

This make it easier 'cheat' on the test (double-click to select). Useful when manually testing generator output.

* docs(test): updated documentation

Clarified start:production

* remove erroneous single quote

* test(travis): revert yarn calls to npm

Travis does not directly support yarn

As described at https://blog.travis-ci.com/2016-11-21-travis-ci-now-supports-yarn
Travis looks for yarn.lock and maps npm calls to yarn, if present.

* config(travis.yaml): move test and build to script section (#1455)

* chore(docs): cleanup of introduction

* chore(docs): cleanup of introduction

* chore(readme): add hitchhikers guide to the readme

* Several minor edits...

* Remove (in no particular order)

* docs(servers): update server conf files for offline-first per #645 (#1381)

* docs(servers): update server conf files for offline-first per #645

re caching see mxstbr#645 (comment)

Apache
http://stackoverflow.com/a/33287352/522756

Nginx
https://www.nginx.com/blog/nginx-caching-guide/

* Update .gitattributes

added "." to nginx.conf

* Remove empty line in list

* Picked a nit :-)

* chore(core): Move mocks/ folder to internals/ (#1460)

* Update CHANGELOG
 e514973

This was referenced Jan 21, 2017

Closed
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment