From cdb9a4ffa82ac56e967fbed397d5917b3055e105 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Nicol=C3=B2=20Ribaudo?= Date: Mon, 25 Nov 2019 16:37:29 -0500 Subject: [PATCH 1/5] chore: replace node by `yarn node` --- Makefile | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/Makefile b/Makefile index e1bc7c939168..1512b10186e6 100644 --- a/Makefile +++ b/Makefile @@ -34,18 +34,18 @@ build-bundle-ci: bootstrap-only $(MAKE) build-bundle generate-standalone: - node packages/babel-standalone/scripts/generate.js + yarn node packages/babel-standalone/scripts/generate.js generate-type-helpers: - node packages/babel-types/scripts/generateTypeHelpers.js + yarn node packages/babel-types/scripts/generateTypeHelpers.js build-typings: build-flow-typings build-typescript-typings build-flow-typings: - node packages/babel-types/scripts/generators/flow.js > packages/babel-types/lib/index.js.flow + yarn node packages/babel-types/scripts/generators/flow.js > packages/babel-types/lib/index.js.flow build-typescript-typings: - node packages/babel-types/scripts/generators/typescript.js > packages/babel-types/lib/index.d.ts + yarn node packages/babel-types/scripts/generators/typescript.js > packages/babel-types/lib/index.d.ts build-standalone: build-babel-standalone build-preset-env-standalone @@ -72,7 +72,7 @@ build-polyfill-dist: build-plugin-transform-runtime-dist: cd packages/babel-plugin-transform-runtime; \ - node scripts/build-dist.js + yarn node scripts/build-dist.js build-no-bundle: clean clean-lib BABEL_ENV=development yarn gulp build-no-bundle @@ -160,13 +160,13 @@ bootstrap-flow: cd build/flow && git checkout $(FLOW_COMMIT) test-flow: - node scripts/parser-tests/flow + yarn node scripts/parser-tests/flow test-flow-ci: build-bundle-ci bootstrap-flow $(MAKE) test-flow test-flow-update-whitelist: - node scripts/parser-tests/flow --update-whitelist + yarn node scripts/parser-tests/flow --update-whitelist bootstrap-typescript: rm -rf ./build/typescript @@ -175,13 +175,13 @@ bootstrap-typescript: cd build/typescript && git checkout $(TYPESCRIPT_COMMIT) test-typescript: - node scripts/parser-tests/typescript + yarn node scripts/parser-tests/typescript test-typescript-ci: build-bundle-ci bootstrap-typescript $(MAKE) test-typescript test-typescript-update-whitelist: - node scripts/parser-tests/typescript --update-whitelist + yarn node scripts/parser-tests/typescript --update-whitelist bootstrap-test262: rm -rf build/test262 @@ -190,13 +190,13 @@ bootstrap-test262: cd build/test262 && git checkout $(TEST262_COMMIT) test-test262: - node scripts/parser-tests/test262 + yarn node scripts/parser-tests/test262 test-test262-ci: build-bundle-ci bootstrap-test262 $(MAKE) test-test262 test-test262-update-whitelist: - node scripts/parser-tests/test262 --update-whitelist + yarn node scripts/parser-tests/test262 --update-whitelist # Does not work on Windows clone-license: @@ -300,7 +300,7 @@ define clean-source-all endef define set-json-field - node -e "\ + yarn node -e "\ require('fs').writeFileSync('$1'.trim(), \ JSON.stringify({ ...require('$1'.trim()), $2: $3 }, null, 2) + '\\n' \ )" From 76fb9b8cb5b11e24f3877e37df164f3d3ba7ea42 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hu=C3=A1ng=20J=C3=B9nli=C3=A0ng?= Date: Thu, 28 Nov 2019 20:18:10 -0500 Subject: [PATCH 2/5] chore: fix tsc command --- scripts/lint-ts-typings.sh | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/scripts/lint-ts-typings.sh b/scripts/lint-ts-typings.sh index 7a5f9193287f..625ff7e375fe 100755 --- a/scripts/lint-ts-typings.sh +++ b/scripts/lint-ts-typings.sh @@ -1,7 +1,6 @@ #!/bin/bash set -e -node="node" tsFlags="--strict" -$node ./node_modules/typescript/bin/tsc $tsFlags ./packages/babel-types/lib/index.d.ts +yarn tsc $tsFlags ./packages/babel-types/lib/index.d.ts From 56d12912f0cedca910fd0b34e3a2c1e4b26457a6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hu=C3=A1ng=20J=C3=B9nli=C3=A0ng?= Date: Fri, 29 Nov 2019 18:12:14 -0500 Subject: [PATCH 3/5] chore: use `yarn` to wrap node --- scripts/test-cov.sh | 3 ++- scripts/test.sh | 6 +++--- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/scripts/test-cov.sh b/scripts/test-cov.sh index 6914ef3fb0f9..5d2aec8e3840 100755 --- a/scripts/test-cov.sh +++ b/scripts/test-cov.sh @@ -1,10 +1,11 @@ #!/bin/bash set -e +node="yarn node" jestArgs="--coverage" if [ -n "$CI" ]; then jestArgs="${jestArgs} --maxWorkers=4 --ci" fi -node_modules/.bin/jest $jestArgs +$node node_modules/.bin/jest $jestArgs diff --git a/scripts/test.sh b/scripts/test.sh index b14189e28a70..ecbd94291614 100755 --- a/scripts/test.sh +++ b/scripts/test.sh @@ -1,11 +1,11 @@ #!/bin/bash set -e -node="node" +node="yarn node" jestArgs=() if [ "$TEST_DEBUG" ]; then - node="node --inspect-brk" + node="$node --inspect-brk" jestArgs+=("--runInBand") fi @@ -23,4 +23,4 @@ if [ -n "$TEST_ONLY" ]; then jestArgs+=("(packages|codemods|eslint)/.*$TEST_ONLY.*/test") fi -$node node_modules/jest/bin/jest.js "${jestArgs[@]}" +$node node_modules/.bin/jest "${jestArgs[@]}" From b941df75dbd19b77ce1b4506b76930f3dd0495c2 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hu=C3=A1ng=20J=C3=B9nli=C3=A0ng?= Date: Fri, 29 Nov 2019 21:07:20 -0500 Subject: [PATCH 4/5] chore: supress yarn run output --- Makefile | 77 ++++++++++++++++++++------------------ scripts/lint-ts-typings.sh | 2 +- scripts/test-cov.sh | 2 +- scripts/test.sh | 2 +- 4 files changed, 43 insertions(+), 40 deletions(-) diff --git a/Makefile b/Makefile index 1512b10186e6..30a5c744f42b 100644 --- a/Makefile +++ b/Makefile @@ -14,6 +14,9 @@ EMPTY := SPACE := $(EMPTY) $(EMPTY) COMMA_SEPARATED_SOURCES = $(subst $(SPACE),$(COMMA),$(SOURCES)) +YARN := yarn --silent +NODE := $(YARN) node + .PHONY: build build-dist watch lint fix clean test-clean test-only test test-ci publish bootstrap @@ -23,10 +26,10 @@ ifneq ("$(BABEL_COVERAGE)", "true") endif build-bundle: clean clean-lib - yarn gulp build + $(YARN) gulp build $(MAKE) generate-standalone generate-type-helpers # call build again as the generated files might need to be compiled again. - yarn gulp build + $(YARN) gulp build $(MAKE) build-typings $(MAKE) build-dist @@ -34,18 +37,18 @@ build-bundle-ci: bootstrap-only $(MAKE) build-bundle generate-standalone: - yarn node packages/babel-standalone/scripts/generate.js + $(NODE) packages/babel-standalone/scripts/generate.js generate-type-helpers: - yarn node packages/babel-types/scripts/generateTypeHelpers.js + $(NODE) packages/babel-types/scripts/generateTypeHelpers.js build-typings: build-flow-typings build-typescript-typings build-flow-typings: - yarn node packages/babel-types/scripts/generators/flow.js > packages/babel-types/lib/index.js.flow + $(NODE) packages/babel-types/scripts/generators/flow.js > packages/babel-types/lib/index.js.flow build-typescript-typings: - yarn node packages/babel-types/scripts/generators/typescript.js > packages/babel-types/lib/index.d.ts + $(NODE) packages/babel-types/scripts/generators/typescript.js > packages/babel-types/lib/index.d.ts build-standalone: build-babel-standalone build-preset-env-standalone @@ -53,16 +56,16 @@ build-standalone-ci: build-bundle-ci $(MAKE) build-standalone build-babel-standalone: - yarn gulp build-babel-standalone + $(YARN) gulp build-babel-standalone build-preset-env-standalone: - yarn gulp build-babel-preset-env-standalone + $(YARN) gulp build-babel-preset-env-standalone prepublish-build-standalone: - BABEL_ENV=production IS_PUBLISH=true yarn gulp build-babel-standalone + BABEL_ENV=production IS_PUBLISH=true $(YARN) gulp build-babel-standalone prepublish-build-preset-env-standalone: - BABEL_ENV=production IS_PUBLISH=true yarn gulp build-babel-preset-env-standalone + BABEL_ENV=production IS_PUBLISH=true $(YARN) gulp build-babel-preset-env-standalone build-dist: build-polyfill-dist build-plugin-transform-runtime-dist @@ -72,17 +75,17 @@ build-polyfill-dist: build-plugin-transform-runtime-dist: cd packages/babel-plugin-transform-runtime; \ - yarn node scripts/build-dist.js + $(NODE) scripts/build-dist.js build-no-bundle: clean clean-lib - BABEL_ENV=development yarn gulp build-no-bundle + BABEL_ENV=development $(YARN) gulp build-no-bundle # Ensure that build artifacts for types are created during local # development too. $(MAKE) generate-type-helpers $(MAKE) build-typings watch: build-no-bundle - BABEL_ENV=development yarn gulp watch + BABEL_ENV=development $(YARN) gulp watch code-quality-ci: flowcheck-ci lint-ci @@ -92,10 +95,10 @@ flowcheck-ci: bootstrap-flowcheck code-quality: flow lint flow: - yarn flow check --strip-root + $(YARN) flow check --strip-root bootstrap-flowcheck: bootstrap-only - yarn gulp build-babel-types + $(YARN) gulp build-babel-types $(MAKE) build-typings lint-ci: lint-js-ci lint-ts-ci @@ -109,7 +112,7 @@ lint-ts-ci: bootstrap-flowcheck lint: lint-js lint-ts lint-js: - yarn eslint scripts $(SOURCES) '*.js' --format=codeframe + $(YARN) eslint scripts $(SOURCES) '*.js' --format=codeframe lint-ts: scripts/lint-ts-typings.sh @@ -117,10 +120,10 @@ lint-ts: fix: fix-json fix-js fix-js: - yarn eslint scripts $(SOURCES) '*.js' --format=codeframe --fix + $(YARN) eslint scripts $(SOURCES) '*.js' --format=codeframe --fix fix-json: - yarn prettier "{$(COMMA_SEPARATED_SOURCES)}/*/test/fixtures/**/options.json" --write --loglevel warn + $(YARN) prettier "{$(COMMA_SEPARATED_SOURCES)}/*/test/fixtures/**/options.json" --write --loglevel warn clean: test-clean rm -f .npmrc @@ -133,7 +136,7 @@ test-clean: $(foreach source, $(SOURCES), \ $(call clean-source-test, $(source))) -# Does not work on Windows; use "yarn jest" instead +# Does not work on Windows; use "$(YARN) jest" instead test-only: BABEL_ENV=test ./scripts/test.sh $(MAKE) test-clean @@ -143,7 +146,7 @@ test: lint test-only test-ci: jest-ci jest-ci: build-standalone-ci - BABEL_ENV=test yarn jest --maxWorkers=4 --ci + BABEL_ENV=test $(YARN) jest --maxWorkers=4 --ci $(MAKE) test-clean # Does not work on Windows @@ -160,13 +163,13 @@ bootstrap-flow: cd build/flow && git checkout $(FLOW_COMMIT) test-flow: - yarn node scripts/parser-tests/flow + $(NODE) scripts/parser-tests/flow test-flow-ci: build-bundle-ci bootstrap-flow $(MAKE) test-flow test-flow-update-whitelist: - yarn node scripts/parser-tests/flow --update-whitelist + $(NODE) scripts/parser-tests/flow --update-whitelist bootstrap-typescript: rm -rf ./build/typescript @@ -175,13 +178,13 @@ bootstrap-typescript: cd build/typescript && git checkout $(TYPESCRIPT_COMMIT) test-typescript: - yarn node scripts/parser-tests/typescript + $(NODE) scripts/parser-tests/typescript test-typescript-ci: build-bundle-ci bootstrap-typescript $(MAKE) test-typescript test-typescript-update-whitelist: - yarn node scripts/parser-tests/typescript --update-whitelist + $(NODE) scripts/parser-tests/typescript --update-whitelist bootstrap-test262: rm -rf build/test262 @@ -190,13 +193,13 @@ bootstrap-test262: cd build/test262 && git checkout $(TEST262_COMMIT) test-test262: - yarn node scripts/parser-tests/test262 + $(NODE) scripts/parser-tests/test262 test-test262-ci: build-bundle-ci bootstrap-test262 $(MAKE) test-test262 test-test262-update-whitelist: - yarn node scripts/parser-tests/test262 --update-whitelist + $(NODE) scripts/parser-tests/test262 --update-whitelist # Does not work on Windows clone-license: @@ -213,11 +216,11 @@ prepublish: new-version: git pull --rebase - yarn lerna version --force-publish=$(FORCE_PUBLISH) + $(YARN) lerna version --force-publish=$(FORCE_PUBLISH) # NOTE: Run make new-version first publish: prepublish - yarn lerna publish from-git + $(YARN) lerna publish from-git $(MAKE) clean publish-ci: prepublish @@ -227,7 +230,7 @@ else echo "Missing NPM_TOKEN env var" exit 1 endif - yarn lerna publish from-git --yes + $(YARN) lerna publish from-git --yes rm -f .npmrc $(MAKE) clean @@ -237,23 +240,23 @@ ifneq ("$(I_AM_USING_VERDACCIO)", "I_AM_SURE") exit 1 endif $(MAKE) prepublish-build - yarn lerna version patch --force-publish=$(FORCE_PUBLISH) --no-push --yes --tag-version-prefix="version-e2e-test-" - yarn lerna publish from-git --registry http://localhost:4873 --yes --tag-version-prefix="version-e2e-test-" + $(YARN) lerna version patch --force-publish=$(FORCE_PUBLISH) --no-push --yes --tag-version-prefix="version-e2e-test-" + $(YARN) lerna publish from-git --registry http://localhost:4873 --yes --tag-version-prefix="version-e2e-test-" $(MAKE) clean publish-eslint: $(call set-json-field, ./eslint/$(PKG)/package.json, private, false) - cd eslint/$(PKG); yarn publish + cd eslint/$(PKG); $(YARN) publish $(call set-json-field, ./eslint/$(PKG)/package.json, private, true) bootstrap-only: lerna-bootstrap -yarn-install: clean-all - yarn --ignore-engines +$(YARN)-install: clean-all + $(YARN) --ignore-engines -lerna-bootstrap: yarn-install +lerna-bootstrap: $(YARN)-install # todo: remove `-- -- --ignore-engines` in Babel 8 - yarn lerna bootstrap -- -- --ignore-engines + $(YARN) lerna bootstrap -- -- --ignore-engines bootstrap: bootstrap-only $(MAKE) build @@ -300,7 +303,7 @@ define clean-source-all endef define set-json-field - yarn node -e "\ + $(NODE) -e "\ require('fs').writeFileSync('$1'.trim(), \ JSON.stringify({ ...require('$1'.trim()), $2: $3 }, null, 2) + '\\n' \ )" diff --git a/scripts/lint-ts-typings.sh b/scripts/lint-ts-typings.sh index 625ff7e375fe..1e308cd44ce8 100755 --- a/scripts/lint-ts-typings.sh +++ b/scripts/lint-ts-typings.sh @@ -3,4 +3,4 @@ set -e tsFlags="--strict" -yarn tsc $tsFlags ./packages/babel-types/lib/index.d.ts +yarn --silent tsc $tsFlags ./packages/babel-types/lib/index.d.ts diff --git a/scripts/test-cov.sh b/scripts/test-cov.sh index 5d2aec8e3840..4d90d4bda385 100755 --- a/scripts/test-cov.sh +++ b/scripts/test-cov.sh @@ -1,7 +1,7 @@ #!/bin/bash set -e -node="yarn node" +node="yarn --silent node" jestArgs="--coverage" if [ -n "$CI" ]; then diff --git a/scripts/test.sh b/scripts/test.sh index ecbd94291614..40109fbe3aa3 100755 --- a/scripts/test.sh +++ b/scripts/test.sh @@ -1,7 +1,7 @@ #!/bin/bash set -e -node="yarn node" +node="yarn --silent node" jestArgs=() if [ "$TEST_DEBUG" ]; then From 46b831e8f3890e238ae3a95145461be136b35ffe Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Hu=C3=A1ng=20J=C3=B9nli=C3=A0ng?= Date: Fri, 29 Nov 2019 22:37:48 -0500 Subject: [PATCH 5/5] chore: disable silent mode for builtin yarn command --- Makefile | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Makefile b/Makefile index 30a5c744f42b..50ff34f61cba 100644 --- a/Makefile +++ b/Makefile @@ -246,15 +246,15 @@ endif publish-eslint: $(call set-json-field, ./eslint/$(PKG)/package.json, private, false) - cd eslint/$(PKG); $(YARN) publish + cd eslint/$(PKG); yarn publish $(call set-json-field, ./eslint/$(PKG)/package.json, private, true) bootstrap-only: lerna-bootstrap -$(YARN)-install: clean-all - $(YARN) --ignore-engines +yarn-install: clean-all + yarn --ignore-engines -lerna-bootstrap: $(YARN)-install +lerna-bootstrap: yarn-install # todo: remove `-- -- --ignore-engines` in Babel 8 $(YARN) lerna bootstrap -- -- --ignore-engines