From 720723a51231ff5674a663574fea7dbb8fc562e9 Mon Sep 17 00:00:00 2001 From: Feiyang1 Date: Thu, 14 Jan 2021 16:05:54 -0800 Subject: [PATCH 1/3] remove prepare from exp packages --- packages-exp/app-compat/package.json | 3 +-- packages-exp/app-exp/package.json | 1 - packages-exp/auth-compat-exp/package.json | 3 +-- packages-exp/auth-exp/package.json | 1 - packages-exp/firebase-exp/package.json | 1 - packages-exp/functions-compat/package.json | 3 +-- packages-exp/functions-exp/package.json | 1 - packages-exp/functions-types-exp/package.json | 2 +- packages-exp/installations-exp/package.json | 1 - packages-exp/performance-exp/package.json | 1 - packages-exp/remote-config-compat/package.json | 3 +-- packages-exp/remote-config-exp/package.json | 1 - repo-scripts/changelog-generator/package.json | 3 +-- 13 files changed, 6 insertions(+), 18 deletions(-) diff --git a/packages-exp/app-compat/package.json b/packages-exp/app-compat/package.json index 8507e7b766f..b98fdb489d8 100644 --- a/packages-exp/app-compat/package.json +++ b/packages-exp/app-compat/package.json @@ -23,8 +23,7 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:browser": "karma start --single-run", "test:browser:debug": "karma start --browsers Chrome --auto-watch", - "test:node": "TS_NODE_FILES=true TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* src/**/*.test.ts --config ../../config/mocharc.node.js", - "prepare": "yarn build:release" + "test:node": "TS_NODE_FILES=true TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* src/**/*.test.ts --config ../../config/mocharc.node.js" }, "license": "Apache-2.0", "dependencies": { diff --git a/packages-exp/app-exp/package.json b/packages-exp/app-exp/package.json index 7c7a36983d8..1a130a20f1a 100644 --- a/packages-exp/app-exp/package.json +++ b/packages-exp/app-exp/package.json @@ -21,7 +21,6 @@ "test:all": "run-p test:browser test:node", "test:browser": "karma start --single-run", "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.ts --config ../../config/mocharc.node.js", - "prepare": "rollup -c rollup.config.release.js && yarn api-report", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs", diff --git a/packages-exp/auth-compat-exp/package.json b/packages-exp/auth-compat-exp/package.json index dae7ea370cd..a7630380d59 100644 --- a/packages-exp/auth-compat-exp/package.json +++ b/packages-exp/auth-compat-exp/package.json @@ -20,8 +20,7 @@ "test:all": "run-p test:browser test:node", "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:browser": "karma start --single-run", - "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.* --config ../../config/mocharc.node.js", - "prepare": "yarn build:release" + "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.* --config ../../config/mocharc.node.js" }, "peerDependencies": { "@firebase/app-compat": "0.x", diff --git a/packages-exp/auth-exp/package.json b/packages-exp/auth-exp/package.json index 51a351b3508..fc3679d7b53 100644 --- a/packages-exp/auth-exp/package.json +++ b/packages-exp/auth-exp/package.json @@ -27,7 +27,6 @@ "test:browser:debug": "karma start --auto-watch", "test:browser:unit:debug": "karma start --auto-watch --unit", "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'src/!(platform_browser|platform_react_native)/**/*.test.ts' --file index.node.ts --config ../../config/mocharc.node.js", - "prepare": "rollup -c rollup.config.release.js", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs", diff --git a/packages-exp/firebase-exp/package.json b/packages-exp/firebase-exp/package.json index 6488dbb8bac..79ced03afa3 100644 --- a/packages-exp/firebase-exp/package.json +++ b/packages-exp/firebase-exp/package.json @@ -32,7 +32,6 @@ "build:compat": "rollup -c compat/rollup.config.js", "build:compat:release": "rollup -c compat/rollup.config.release.js", "dev": "rollup -c -w", - "prepare": "yarn build:release", "test": "echo 'No test suite for firebase wrapper'", "test:ci": "echo 'No test suite for firebase wrapper'" }, diff --git a/packages-exp/functions-compat/package.json b/packages-exp/functions-compat/package.json index bfcbfbc05a1..b4b2f5bab9a 100644 --- a/packages-exp/functions-compat/package.json +++ b/packages-exp/functions-compat/package.json @@ -41,8 +41,7 @@ "test:browser": "karma start --single-run", "test:browser:debug": "karma start --browsers=Chrome --auto-watch", "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'src/{,!(browser)/**/}*.test.ts' --file src/index.node.ts --config ../../config/mocharc.node.js", - "test:emulator": "env FIREBASE_FUNCTIONS_HOST=http://localhost FIREBASE_FUNCTIONS_PORT=5005 run-p test:node", - "prepare": "yarn build:release" + "test:emulator": "env FIREBASE_FUNCTIONS_HOST=http://localhost FIREBASE_FUNCTIONS_PORT=5005 run-p test:node" }, "typings": "dist/functions-compat-public.d.ts", "dependencies": { diff --git a/packages-exp/functions-exp/package.json b/packages-exp/functions-exp/package.json index 083d15b3e62..60dc296c658 100644 --- a/packages-exp/functions-exp/package.json +++ b/packages-exp/functions-exp/package.json @@ -23,7 +23,6 @@ "test:browser:debug": "karma start --browsers=Chrome --auto-watch", "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'src/{,!(browser)/**/}*.test.ts' --file src/index.node.ts --config ../../config/mocharc.node.js", "test:emulator": "env FIREBASE_FUNCTIONS_EMULATOR_ORIGIN=http://localhost:5005 run-p test:node", - "prepare": "yarn build:release", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs", diff --git a/packages-exp/functions-types-exp/package.json b/packages-exp/functions-types-exp/package.json index 2d385c5c0dd..f9b3cb02418 100644 --- a/packages-exp/functions-types-exp/package.json +++ b/packages-exp/functions-types-exp/package.json @@ -8,7 +8,7 @@ "scripts": { "test": "tsc", "test:ci": "node ../../scripts/run_tests_in_ci.js", - "prepare": "node ../../scripts/exp/remove-exp.js ./index.d.ts", + "build:release": "node ../../scripts/exp/remove-exp.js ./index.d.ts", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs", diff --git a/packages-exp/installations-exp/package.json b/packages-exp/installations-exp/package.json index 16d64e4c1dd..7ea6761c711 100644 --- a/packages-exp/installations-exp/package.json +++ b/packages-exp/installations-exp/package.json @@ -25,7 +25,6 @@ "serve": "yarn serve:build && yarn serve:host", "serve:build": "rollup -c test-app/rollup.config.js", "serve:host": "http-server -c-1 test-app", - "prepare": "yarn build:release", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs", diff --git a/packages-exp/performance-exp/package.json b/packages-exp/performance-exp/package.json index 2b2c50b4183..591f9f8ec0f 100644 --- a/packages-exp/performance-exp/package.json +++ b/packages-exp/performance-exp/package.json @@ -20,7 +20,6 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:browser", "test:browser": "karma start --single-run", "test:debug": "karma start --browsers=Chrome --auto-watch", - "prepare": "yarn build:release", "prettier": "prettier --write '{src,test}/**/*.{js,ts}'", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", diff --git a/packages-exp/remote-config-compat/package.json b/packages-exp/remote-config-compat/package.json index 8e3da2189e1..ae6bea83c0c 100644 --- a/packages-exp/remote-config-compat/package.json +++ b/packages-exp/remote-config-compat/package.json @@ -20,8 +20,7 @@ "test:all": "run-p test:browser", "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:browser": "karma start --single-run", - "test:browser:debug": "karma start --browsers Chrome --auto-watch", - "prepare": "yarn build:release" + "test:browser:debug": "karma start --browsers Chrome --auto-watch" }, "license": "Apache-2.0", "peerDependencies": { diff --git a/packages-exp/remote-config-exp/package.json b/packages-exp/remote-config-exp/package.json index 0d8a6df2e85..354b394266d 100644 --- a/packages-exp/remote-config-exp/package.json +++ b/packages-exp/remote-config-exp/package.json @@ -21,7 +21,6 @@ "test:browser": "karma start --single-run", "test:debug": "karma start --browsers=Chrome --auto-watch", "prettier": "prettier --write '{src,test}/**/*.{js,ts}'", - "prepare": "yarn build:release", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs", diff --git a/repo-scripts/changelog-generator/package.json b/repo-scripts/changelog-generator/package.json index 3bbb73d11d2..808e1ed3b1c 100644 --- a/repo-scripts/changelog-generator/package.json +++ b/repo-scripts/changelog-generator/package.json @@ -13,8 +13,7 @@ "lint:fix": "eslint --fix -c .eslintrc.js '**/*.ts' --ignore-path '../../.gitignore'", "build": "tsc", "build:dev": "tsc -w", - "test": "tsc -p . --noEmit", - "prepare": "yarn build" + "test": "tsc -p . --noEmit" }, "dependencies": { "@changesets/types": "3.2.0", From ea4ea5443f53ba968742880eeff1e9626bac17aa Mon Sep 17 00:00:00 2001 From: Feiyang1 Date: Thu, 14 Jan 2021 16:11:30 -0800 Subject: [PATCH 2/3] remove prepare from packages --- packages-exp/installations-compat/package.json | 1 - packages-exp/messaging-exp/package.json | 1 - packages/analytics/package.json | 3 +-- packages/app/package.json | 3 +-- packages/auth/package.json | 1 - packages/component/package.json | 3 +-- packages/database/package.json | 3 +-- packages/firebase/package.json | 1 - packages/functions/package.json | 3 +-- packages/installations/package.json | 3 +-- packages/logger/package.json | 3 +-- packages/messaging/package.json | 3 +-- packages/performance/package.json | 1 - packages/polyfill/package.json | 3 +-- packages/remote-config/package.json | 3 +-- packages/rules-unit-testing/package.json | 3 +-- packages/rxfire/package.json | 1 - packages/storage/package.json | 1 - packages/template/package.json | 3 +-- packages/util/package.json | 3 +-- packages/webchannel-wrapper/package.json | 1 - 21 files changed, 13 insertions(+), 34 deletions(-) diff --git a/packages-exp/installations-compat/package.json b/packages-exp/installations-compat/package.json index 7423f473af5..61adef7a637 100644 --- a/packages-exp/installations-compat/package.json +++ b/packages-exp/installations-compat/package.json @@ -25,7 +25,6 @@ "serve": "yarn serve:build && yarn serve:host", "serve:build": "rollup -c test-app/rollup.config.js", "serve:host": "http-server -c-1 test-app", - "prepare": "yarn build:release", "api-report": "api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs", diff --git a/packages-exp/messaging-exp/package.json b/packages-exp/messaging-exp/package.json index 1416ac91661..d14d1df121e 100644 --- a/packages-exp/messaging-exp/package.json +++ b/packages-exp/messaging-exp/package.json @@ -22,7 +22,6 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js", "test:karma": "karma start --single-run", "test:debug": "karma start --browsers=Chrome --auto-watch", - "prepare": "yarn build", "api-report": "api-extractor run --local --verbose", "type-check": "tsc --noEmit" }, diff --git a/packages/analytics/package.json b/packages/analytics/package.json index 30add5f9179..b6cba0a4535 100644 --- a/packages/analytics/package.json +++ b/packages/analytics/package.json @@ -17,8 +17,7 @@ "test:all": "run-p test:browser test:integration", "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:browser": "karma start --single-run --nocache", - "test:integration": "karma start ./karma.integration.conf.js --single-run --nocache", - "prepare": "yarn build" + "test:integration": "karma start ./karma.integration.conf.js --single-run --nocache" }, "peerDependencies": { "@firebase/app": "0.x", diff --git a/packages/app/package.json b/packages/app/package.json index bdd8e1bee94..881544efa49 100644 --- a/packages/app/package.json +++ b/packages/app/package.json @@ -22,8 +22,7 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:browser": "karma start --single-run", "test:browser:debug": "karma start --browsers Chrome --auto-watch", - "test:node": "TS_NODE_FILES=true TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* --config ../../config/mocharc.node.js", - "prepare": "yarn build" + "test:node": "TS_NODE_FILES=true TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* --config ../../config/mocharc.node.js" }, "license": "Apache-2.0", "dependencies": { diff --git a/packages/auth/package.json b/packages/auth/package.json index b574515f49e..e11ca56f9e0 100644 --- a/packages/auth/package.json +++ b/packages/auth/package.json @@ -15,7 +15,6 @@ "build:deps": "lerna run --scope @firebase/auth --include-dependencies build", "demo": "./buildtools/run_demo.sh", "generate-test-files": "./buildtools/generate_test_files.sh", - "prepare": "yarn build", "serve": "yarn build && yarn generate-test-files && gulp serve", "test": "yarn generate-test-files && ./buildtools/run_tests.sh", "test:ci": "node ../../scripts/run_tests_in_ci.js" diff --git a/packages/component/package.json b/packages/component/package.json index 55f2e218b52..e87d004140b 100644 --- a/packages/component/package.json +++ b/packages/component/package.json @@ -18,8 +18,7 @@ "test:all": "run-p test:browser test:node", "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:browser": "karma start --single-run", - "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.ts --config ../../config/mocharc.node.js", - "prepare": "yarn build" + "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.ts --config ../../config/mocharc.node.js" }, "dependencies": { "@firebase/util": "0.3.4", diff --git a/packages/database/package.json b/packages/database/package.json index 6b55deca966..1053a1d23f6 100644 --- a/packages/database/package.json +++ b/packages/database/package.json @@ -21,8 +21,7 @@ "test:all": "run-p lint test:browser test:node", "test:browser": "karma start --single-run", "test:node": "TS_NODE_FILES=true TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'test/{,!(browser)/**/}*.test.ts' --file index.node.ts --config ../../config/mocharc.node.js", - "test:emulator": "ts-node --compiler-options='{\"module\":\"commonjs\"}' ../../scripts/emulator-testing/database-test-runner.ts", - "prepare": "yarn build" + "test:emulator": "ts-node --compiler-options='{\"module\":\"commonjs\"}' ../../scripts/emulator-testing/database-test-runner.ts" }, "license": "Apache-2.0", "peerDependencies": {}, diff --git a/packages/firebase/package.json b/packages/firebase/package.json index d17ffcc560e..4e842c3192f 100644 --- a/packages/firebase/package.json +++ b/packages/firebase/package.json @@ -36,7 +36,6 @@ "build": "rollup -c", "build:deps": "lerna run --scope firebase --include-dependencies build", "dev": "rollup -c -w", - "prepare": "yarn build", "test": "echo 'No test suite for firebase wrapper'", "test:ci": "echo 'No test suite for firebase wrapper'" }, diff --git a/packages/functions/package.json b/packages/functions/package.json index 6cdff663b21..3f2f4df7acf 100644 --- a/packages/functions/package.json +++ b/packages/functions/package.json @@ -20,8 +20,7 @@ "test:browser": "karma start --single-run", "test:browser:debug": "karma start --browsers=Chrome --auto-watch", "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'test/{,!(browser)/**/}*.test.ts' --file index.node.ts --config ../../config/mocharc.node.js", - "test:emulator": "env FIREBASE_FUNCTIONS_EMULATOR_ORIGIN=http://localhost:5005 run-p test:node", - "prepare": "yarn build" + "test:emulator": "env FIREBASE_FUNCTIONS_EMULATOR_ORIGIN=http://localhost:5005 run-p test:node" }, "license": "Apache-2.0", "peerDependencies": { diff --git a/packages/installations/package.json b/packages/installations/package.json index 872dae2319c..01d14d1f650 100644 --- a/packages/installations/package.json +++ b/packages/installations/package.json @@ -18,8 +18,7 @@ "test:debug": "karma start --browsers=Chrome --auto-watch", "serve": "yarn serve:build && yarn serve:host", "serve:build": "rollup -c test-app/rollup.config.js", - "serve:host": "http-server -c-1 test-app", - "prepare": "yarn build" + "serve:host": "http-server -c-1 test-app" }, "repository": { "directory": "packages/installations", diff --git a/packages/logger/package.json b/packages/logger/package.json index 7633c5f79e4..4ccc178b3dc 100644 --- a/packages/logger/package.json +++ b/packages/logger/package.json @@ -18,8 +18,7 @@ "test:all": "run-p test:browser test:node", "test:browser": "karma start --single-run", "test:browser:debug": "karma start --browsers Chrome --auto-watch", - "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* --config ../../config/mocharc.node.js", - "prepare": "yarn build" + "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* --config ../../config/mocharc.node.js" }, "license": "Apache-2.0", "devDependencies": { diff --git a/packages/messaging/package.json b/packages/messaging/package.json index 72e4ea25972..7f573795b8b 100644 --- a/packages/messaging/package.json +++ b/packages/messaging/package.json @@ -17,8 +17,7 @@ "test:integration": "test:karma && cd ../../integration/messaging && npm run-script test", "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:karma", "test:karma": "karma start --single-run", - "test:debug": "karma start --browsers=Chrome --auto-watch", - "prepare": "yarn build" + "test:debug": "karma start --browsers=Chrome --auto-watch" }, "license": "Apache-2.0", "peerDependencies": { diff --git a/packages/performance/package.json b/packages/performance/package.json index fda55e7e345..761f33b9f15 100644 --- a/packages/performance/package.json +++ b/packages/performance/package.json @@ -18,7 +18,6 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:browser", "test:browser": "karma start --single-run", "test:debug": "karma start --browsers=Chrome --auto-watch", - "prepare": "yarn build", "prettier": "prettier --write '{src,test}/**/*.{js,ts}'" }, "peerDependencies": { diff --git a/packages/polyfill/package.json b/packages/polyfill/package.json index 8c9b3a34aa9..7d552a03816 100644 --- a/packages/polyfill/package.json +++ b/packages/polyfill/package.json @@ -12,8 +12,7 @@ "build": "rollup -c", "dev": "rollup -c -w", "test": "echo 'No test suite for polyfills'", - "test:ci": "echo 'No test suite for polyfills'", - "prepare": "yarn build" + "test:ci": "echo 'No test suite for polyfills'" }, "license": "Apache-2.0", "dependencies": { diff --git a/packages/remote-config/package.json b/packages/remote-config/package.json index 98897278ef4..25d54788cc4 100644 --- a/packages/remote-config/package.json +++ b/packages/remote-config/package.json @@ -18,8 +18,7 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:browser", "test:browser": "karma start --single-run", "test:debug": "karma start --browsers=Chrome --auto-watch", - "prettier": "prettier --write '{src,test}/**/*.{js,ts}'", - "prepare": "yarn build" + "prettier": "prettier --write '{src,test}/**/*.{js,ts}'" }, "peerDependencies": { "@firebase/app": "0.x", diff --git a/packages/rules-unit-testing/package.json b/packages/rules-unit-testing/package.json index 43b02d87c14..2097d4e495e 100644 --- a/packages/rules-unit-testing/package.json +++ b/packages/rules-unit-testing/package.json @@ -14,8 +14,7 @@ "dev": "rollup -c -w", "test:nyc": "TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha 'test/{,!(browser)/**/}*.test.ts' --config ../../config/mocharc.node.js", "test": "firebase --project=foo --debug emulators:exec 'yarn test:nyc'", - "test:ci": "node ../../scripts/run_tests_in_ci.js -s test", - "prepare": "yarn build" + "test:ci": "node ../../scripts/run_tests_in_ci.js -s test" }, "license": "Apache-2.0", "dependencies": { diff --git a/packages/rxfire/package.json b/packages/rxfire/package.json index 33791839f9a..75ee8eba159 100644 --- a/packages/rxfire/package.json +++ b/packages/rxfire/package.json @@ -26,7 +26,6 @@ "build": "rollup -c", "build:deps": "lerna run --scope rxfire --include-dependencies build", "dev": "rollup -c -w", - "prepare": "yarn build", "test": "run-p lint test:browser", "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:browser", "test:browser": "karma start --single-run", diff --git a/packages/storage/package.json b/packages/storage/package.json index 6d134c14ea7..94e5cb3fca8 100644 --- a/packages/storage/package.json +++ b/packages/storage/package.json @@ -26,7 +26,6 @@ "test:browser:compat": "karma start --single-run --compat", "test:browser:exp": "karma start --single-run --exp", "test:browser": "karma start --single-run", - "prepare": "yarn build", "prettier": "prettier --write 'src/**/*.ts' 'test/**/*.ts'", "api-report": "api-extractor run --local --verbose" }, diff --git a/packages/template/package.json b/packages/template/package.json index 1ade0e82ba8..8bd1583d7dd 100644 --- a/packages/template/package.json +++ b/packages/template/package.json @@ -19,8 +19,7 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:all": "run-p test:browser test:node", "test:browser": "karma start --single-run", - "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.* --config ../../config/mocharc.node.js", - "prepare": "yarn build" + "test:node": "TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha src/**/*.test.* --config ../../config/mocharc.node.js" }, "peerDependencies": { "@firebase/app": "0.x", diff --git a/packages/util/package.json b/packages/util/package.json index 53a64f999fb..eff43f1973f 100644 --- a/packages/util/package.json +++ b/packages/util/package.json @@ -18,8 +18,7 @@ "test:ci": "node ../../scripts/run_tests_in_ci.js -s test:all", "test:all": "run-p test:browser test:node", "test:browser": "karma start --single-run", - "test:node": "TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* --config ../../config/mocharc.node.js", - "prepare": "yarn build" + "test:node": "TS_NODE_CACHE=NO TS_NODE_COMPILER_OPTIONS='{\"module\":\"commonjs\"}' nyc --reporter lcovonly -- mocha test/**/*.test.* --config ../../config/mocharc.node.js" }, "license": "Apache-2.0", "dependencies": { diff --git a/packages/webchannel-wrapper/package.json b/packages/webchannel-wrapper/package.json index cd708726f7d..fce08a7dece 100644 --- a/packages/webchannel-wrapper/package.json +++ b/packages/webchannel-wrapper/package.json @@ -10,7 +10,6 @@ "scripts": { "dev": "watch 'yarn build' src", "build": "gulp", - "prepare": "yarn build", "test": "echo 'No test suite for webchannel-wrapper'", "test:ci": "echo 'No test suite for webchannel-wrapper'" }, From bc6bc56eaedfa83862fad10a309ea59aca1dd41c Mon Sep 17 00:00:00 2001 From: Feiyang1 Date: Thu, 14 Jan 2021 16:33:24 -0800 Subject: [PATCH 3/3] cleanup --- package.json | 3 +-- packages/auth/buildtools/run_demo.sh | 2 +- packages/firestore/package.json | 1 - scripts/exp/prepare-firestore-for-exp-release.ts | 2 -- 4 files changed, 2 insertions(+), 6 deletions(-) diff --git a/package.json b/package.json index 15343b5860d..f16a72df00b 100644 --- a/package.json +++ b/package.json @@ -24,8 +24,7 @@ "dev": "lerna run --parallel --scope @firebase/* --scope firebase --scope rxfire dev", "build": "lerna run --scope @firebase/app-exp build:deps && lerna run --scope @firebase/* --scope firebase --scope rxfire --ignore @firebase/app-exp build", "build:exp": "lerna run --scope @firebase/*-exp --scope @firebase/*-compat --scope firebase-exp build", - "build:release": "lerna run --scope @firebase/app-exp build:deps && lerna run --scope @firebase/* --scope firebase --scope rxfire --ignore @firebase/*-exp --ignore @firebase/*-compat prepare", - "build:exp:release": "yarn --cwd packages/app build:deps && lerna run --scope @firebase/*-exp --scope @firebase/*-compat --scope firebase-exp prepare && yarn --cwd packages-exp/app-exp typings:public", + "build:release": "lerna run --scope @firebase/app-exp build:deps && lerna run --scope @firebase/* --scope firebase --scope rxfire --ignore @firebase/*-exp --ignore @firebase/*-compat build", "build:changed": "ts-node-script scripts/ci-test/build_changed.ts", "link:packages": "lerna exec --scope @firebase/* --scope firebase --scope rxfire -- yarn link", "stage:packages": "./scripts/prepublish.sh", diff --git a/packages/auth/buildtools/run_demo.sh b/packages/auth/buildtools/run_demo.sh index 43c151af518..0f5feaafb3c 100755 --- a/packages/auth/buildtools/run_demo.sh +++ b/packages/auth/buildtools/run_demo.sh @@ -23,7 +23,7 @@ cd "$(dirname $(dirname "$0"))" # Go back to repo root and build all binaries needed for the demo app. cd ../.. -yarn prepare +yarn build:release # Go back to Auth package. cd packages/auth # Make dist directory if not already there. diff --git a/packages/firestore/package.json b/packages/firestore/package.json index 41b675acf01..de4e026cdc0 100644 --- a/packages/firestore/package.json +++ b/packages/firestore/package.json @@ -46,7 +46,6 @@ "test:node:persistence:prod": "node ./scripts/run-tests.js --main=index.node.ts --persistence 'test/{,!(browser|lite)/**/}*.test.ts'", "test:travis": "ts-node --compiler-options='{\"module\":\"commonjs\"}' ../../scripts/emulator-testing/firestore-test-runner.ts", "test:minified": "(cd ../../integration/firestore ; yarn test)", - "prepare": "yarn build:release", "api-report": "rm -rf temp && api-extractor run --local --verbose", "predoc": "node ../../scripts/exp/remove-exp.js temp", "doc": "api-documenter markdown --input temp --output docs" diff --git a/scripts/exp/prepare-firestore-for-exp-release.ts b/scripts/exp/prepare-firestore-for-exp-release.ts index 166e9c95997..567233ec2f7 100644 --- a/scripts/exp/prepare-firestore-for-exp-release.ts +++ b/scripts/exp/prepare-firestore-for-exp-release.ts @@ -60,8 +60,6 @@ export async function prepare() { packageJson.typings = expPackageJson.typings.replace('../', ''); - delete packageJson.scripts.prepare; - // include files to be published packageJson.files = [ ...packageJson.files,