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

Update to create-react-app v3.2.0 tag #5

Open
wants to merge 44 commits into
base: master
Choose a base branch
from

Conversation

halex2005
Copy link

In this PR I've squashed your 4 commits to one and have rebased them to create-create-app v3.2.0 tag.

Could you release it to npm?
Do you have any CI?
Could we catch regress automatically if something rebased wrong?

xiaoxiangmoe and others added 30 commits August 23, 2019 09:20
cleanup is called now by default after each test if the testing framework supports the afterEach global
* Adopt Contributor Covenant

* Add link to Contributor Covenant to docs site
- use ESLint cache
- remove useless `watch` option in fork-ts-checker
Add blog post that takes a React developer step-by-step through many features of AWS. Blog post is suitable for a beginner to follow and have a production ready workflow.
rakannimer and others added 14 commits September 30, 2019 10:49
* Added the alias for profiling and output change to keep the classNames and functionNames for human readbility

* defined isEnvProductionProfile with other isEnv checks

* moved the keep_classnames and keep_fnames to terserOptions scope

* resolve merge conflict for yarn.lock.cache

* revert yarn.lock.cache to master yarn.lock.cache
- git checkout origin/master -- packages/create-react-app/yarn.lock.cached

* Comment and Boolean Check
- I clarified the comment and specified the use case
- Changed the environment check to check for the specific true rather than
the assumed primitive value as before.

* Replaced env with flag
- Per suggestion --profile flag used instead of env variable PROFILE_APP

* documentation in available scripts section with suggested information

* resolved a local git issue. Fixed documentation error.

* moved documentation to suggested file
- Added a brief summary of profiling in available scripts section.
The summary references the production-build document. Which is the
file I moved the new documentation into under a new Header for production support.

* Update production-build.md


Co-authored-by: Ian Sutherland <[email protected]>
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.