Reset build script#2767
Merged
compulim merged 36 commits intomicrosoft:masterfrom Jan 15, 2020
Merged
Conversation
Contributor
|
what about this? #2394 |
Contributor
|
Are we closing #2394 in this PR too? could you add to Fix line in top comment? |
corinagum
reviewed
Jan 10, 2020
corinagum
suggested changes
Jan 13, 2020
corinagum
reviewed
Jan 14, 2020
corinagum
reviewed
Jan 14, 2020
corinagum
reviewed
Jan 14, 2020
corinagum
reviewed
Jan 14, 2020
corinagum
approved these changes
Jan 14, 2020
Contributor
corinagum
left a comment
There was a problem hiding this comment.
LGTM! As a low priority I would like to go through all our .md files to match up tone and tense, etc. xD
Co-Authored-By: Corina <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR will need to update CI pipeline.
Changelog Entry
Changed
[email protected], by @compulim in PR #2767Description
npm installtime by about 4xnpm startwill start dev modeservemanually)npm prepublishOnlywill build production bitsDesign
For all packages, we provide 3 favors and 2 build scripts.
Favors
We offer 3 build favors:
Tests run locally will be using development build without any code coverage collection.
To select different build favors, set
node_envenvironment variable to:productiontestdevelopmentor not setScripts
We offer 2 types of build processes:
npm run buildwill build oncenpm startwill build continuously with watchSpecific Changes
npm linkin samples, Lerna has no values in samples projectnpm run bootstrapwill always use CI (CI = nuke + faster install)npm run buildto build oncenpm startto build continuouslycleanand most scriptsnpm testwill target for development mode (watch, no coverage)npm run test:ciwill be used on VSTS (no watch, but coverage).babelrctobabel.config.jsonwebpack*.config.jsinto a singlewebpack.config.jsnode_envto determines build favor (development, production, testing)tsconfig.json, we are not usingtscto build, only use for type checking and emit declarationsprettier.ymlat root for prettifyingbabel-jest-config.jsand other config files.travis.ymlto match VSTSazure-pipelines.yml(we will tune and enable it after merged)BUILD_SCRIPTS.mdfor the build script design and verificationpackages/README.mdfor explanation of different packagesCONTRIBUTING.md.dockerignoreto improve Docker build timemain()in console)jest.config.jsfilenpm testdirectlinespeech.development.jswithserveTo-do
.babelrc.jsback tobabel.config.jsonbabel-jest-config.js/.github/CONTRIBUTING.md/packages/README.md