Skip to content

Commit d1d33a4

Browse files
Fixing inconsistencies in package.json scripts (Azure#13936)
1 parent 51bd3e0 commit d1d33a4

File tree

18 files changed

+76
-52
lines changed

18 files changed

+76
-52
lines changed

sdk/appconfiguration/app-configuration/rollup.base.config.js

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -141,7 +141,11 @@ export function browserConfig(test = false) {
141141
baseConfig.onwarn = ignoreKnownWarnings;
142142

143143
if (test) {
144-
baseConfig.input = ["dist-esm/test/*.spec.js", "dist-esm/test/internal/*.spec.js"];
144+
baseConfig.input = [
145+
"dist-esm/test/*.spec.js",
146+
"dist-esm/test/internal/*.spec.js",
147+
"dist-esm/test/public/*.spec.js"
148+
];
145149

146150
baseConfig.external.unshift(...["process"]);
147151

sdk/core/abort-controller/package.json

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,14 @@
77
"module": "dist-esm/src/index.js",
88
"scripts": {
99
"audit": "node ../../../common/scripts/rush-audit.js && rimraf node_modules package-lock.json && npm i --package-lock-only 2>&1 && npm audit",
10-
"build:es6": "tsc -p tsconfig.json",
11-
"build:nodebrowser": "rollup -c 2>&1",
1210
"build:types": "downlevel-dts types/src types/3.1",
13-
"build:test": "rollup -c rollup.test.config.js 2>&1",
14-
"build": "npm run build:es6 && npm run build:nodebrowser && npm run build:types && npm run extract-api",
11+
"build:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c 2>&1 && npm run extract-api",
12+
"build:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c 2>&1",
13+
"build:nodebrowser": "rollup -c 2>&1",
14+
"build:test": "tsc -p . && rollup -c rollup.test.config.js 2>&1",
15+
"build:test:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c rollup.test.config.js 2>&1",
16+
"build:test:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c rollup.test.config.js 2>&1",
17+
"build": "tsc -p . && npm run build:nodebrowser && npm run build:types",
1518
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
1619
"clean": "rimraf dist dist-esm dist-test types temp dist-browser/*.js* dist-browser/*.zip statistics.html coverage coverage-browser .nyc_output *.tgz *.log test*.xml TEST*.xml",
1720
"execute:samples": "echo skipped",

sdk/core/core-lro/package.json

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -65,11 +65,14 @@
6565
},
6666
"scripts": {
6767
"audit": "node ../../../common/scripts/rush-audit.js && rimraf node_modules package-lock.json && npm i --package-lock-only 2>&1 && npm audit",
68-
"build": "npm run build:es6 && npm run build:nodebrowser && npm run build:samples && npm run extract-api",
68+
"build": "tsc -p . && npm run build:nodebrowser && api-extractor run --local",
6969
"build:samples": "tsc -p tsconfig.samples.json",
70-
"build:es6": "tsc -p tsconfig.json",
7170
"build:nodebrowser": "rollup -c 2>&1",
72-
"build:test": "npm run build:es6 && rollup -c rollup.test.config.js 2>&1",
71+
"build:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c 2>&1 && npm run extract-api",
72+
"build:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c 2>&1",
73+
"build:test": "tsc -p . && rollup -c rollup.test.config.js 2>&1",
74+
"build:test:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c rollup.test.config.js 2>&1",
75+
"build:test:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c rollup.test.config.js 2>&1",
7376
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"*.{js,json}\" \"test/**/*.ts\" \"samples/**/*.ts\"",
7477
"clean": "rimraf dist dist-esm dist-test types *.log browser statistics.html coverage src/**/*.js test/**/*.js",
7578
"execute:samples": "echo skipped",
@@ -80,12 +83,12 @@
8083
"integration-test": "npm run integration-test:node && npm run integration-test:browser",
8184
"lint:fix": "eslint package.json api-extractor.json src test --ext .ts --fix --fix-type [problem,suggestion]",
8285
"lint": "eslint package.json api-extractor.json src test --ext .ts",
83-
"test:browser": "npm run build:test && npm run unit-test:browser && npm run integration-test:browser",
84-
"test:node": "npm run build:test && npm run unit-test:node && npm run integration-test:node",
86+
"test:browser": "npm run build:test:browser && npm run unit-test:browser && npm run integration-test:browser",
87+
"test:node": "npm run build:test:node && npm run unit-test:node && npm run integration-test:node",
8588
"test": "npm run build:test && npm run unit-test",
8689
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
87-
"unit-test:browser": "npm run build:test && karma start --single-run",
88-
"unit-test:node": "npm run build:test && nyc mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 1200000 --full-trace dist-test/index.node.js",
90+
"unit-test:browser": "karma start --single-run",
91+
"unit-test:node": "nyc mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 1200000 --full-trace dist-test/index.node.js",
8992
"pack": "npm pack 2>&1",
9093
"prebuild": "npm run clean",
9194
"docs": "typedoc --excludePrivate --excludeNotExported --excludeExternals --stripInternal --mode file --out ./dist/docs ./src"
@@ -110,6 +113,7 @@
110113
"@types/node": "^8.0.0",
111114
"assert": "^1.4.1",
112115
"chai": "^4.2.0",
116+
"cross-env": "^7.0.2",
113117
"eslint": "^7.15.0",
114118
"karma": "^5.1.0",
115119
"karma-chrome-launcher": "^3.0.0",

sdk/digitaltwins/digital-twins-core/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,10 +15,10 @@
1515
"audit": "node ../../../common/scripts/rush-audit.js && rimraf node_modules package-lock.json && npm i --package-lock-only 2>&1 && npm audit",
1616
"build:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c 2>&1",
1717
"build:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c 2>&1 && npm run extract-api",
18-
"build": "npm run build:node && npm run build:browser",
18+
"build": "tsc -p . && rollup -c 2>&1 && api-extractor run --local",
1919
"build:test:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c rollup.test.config.js 2>&1",
2020
"build:test:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c rollup.test.config.js 2>&1",
21-
"build:test": "npm run build:test:node && npm run build:test:browser",
21+
"build:test": "tsc -p . && rollup -c rollup.test.config.js 2>&1",
2222
"build:samples": "echo Skipped.",
2323
"check-format": "prettier --list-different --config ../../.prettierrc.json --ignore-path ../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
2424
"clean": "rimraf dist dist-esm dist-browser test-dist test-browser types *.tgz *.log",

sdk/eventgrid/eventgrid/package.json

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,7 @@
5656
"build": "tsc -p . && rollup -c 2>&1 && api-extractor run --local",
5757
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
5858
"clean": "rimraf dist dist-browser dist-esm test-dist temp types *.tgz *.log",
59-
"execute:js-samples": "node ../../../common/scripts/run-samples.js samples/javascript/",
60-
"execute:ts-samples": "node ../../../common/scripts/run-samples.js samples/typescript/dist/samples/typescript/src/",
61-
"execute:samples": "npm run build:samples && npm run execute:js-samples && npm run execute:ts-samples",
59+
"execute:samples": "npm run build:samples && dev-tool samples run samples/javascript samples/typescript/dist/samples/typescript/src/",
6260
"extract-api": "tsc -p . && api-extractor run --local",
6361
"format": "prettier --write --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
6462
"integration-test:browser": "karma start --single-run",

sdk/eventhub/event-hubs/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@
5151
"build:test:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c rollup.test.config.js 2>&1",
5252
"build:test": "tsc -p . && rollup -c rollup.test.config.js 2>&1",
5353
"build:types": "downlevel-dts types/latest types/3.1",
54-
"build": "tsc -p . && rollup -c 2>&1 && npm run extract-api && npm run build:types",
54+
"build": "tsc -p . && rollup -c 2>&1 && api-extractor run --local && npm run build:types",
5555
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
5656
"clean": "rimraf dist dist-esm test-dist types *.tgz *.log",
5757
"execute:samples": "npm run build:samples && echo Skipped.",

sdk/eventhub/event-processor-host/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@
3737
"build:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c 2>&1",
3838
"build:samples": "dev-tool samples prep && cd dist-samples && tsc",
3939
"build:test": "tsc -p . && cross-env ONLY_NODE=true rollup -c rollup.test.config.js 2>&1",
40-
"build": "tsc -p . && rollup -c 2>&1 && npm run extract-api",
40+
"build": "tsc -p . && rollup -c 2>&1 && api-extractor run --local",
4141
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
4242
"clean": "rimraf dist dist-esm test-dist typings *.tgz *.log",
4343
"execute:samples": "echo skipped",

sdk/eventhub/eventhubs-checkpointstore-blob/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@
3838
"build:test:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c rollup.test.config.js 2>&1",
3939
"build:test:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c rollup.test.config.js 2>&1",
4040
"build:test": "npm run build:test:node",
41-
"build": "tsc -p . && rollup -c 2>&1 && npm run extract-api",
41+
"build": "tsc -p . && rollup -c 2>&1 && api-extractor run --local",
4242
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
4343
"clean": "rimraf dist dist-esm test-dist typings *.tgz *.log",
4444
"execute:samples": "echo skipped",

sdk/keyvault/keyvault-admin/package.json

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -47,15 +47,14 @@
4747
"scripts": {
4848
"audit": "node ../../../common/scripts/rush-audit.js && rimraf node_modules package-lock.json && npm i --package-lock-only 2>&1 && npm audit",
4949
"build:samples": "echo skipped",
50-
"build:es6": "tsc -p tsconfig.json",
50+
"build:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c 2>&1",
51+
"build:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c 2>&1",
5152
"build:nodebrowser": "rollup -c 2>&1",
52-
"build:test": "npm run build:es6 && rollup -c rollup.test.config.js 2>&1",
53-
"build": "npm run extract-api && npm run build:es6 && npm run build:nodebrowser",
53+
"build:test": "tsc -p . && rollup -c rollup.test.config.js 2>&1",
54+
"build": "tsc -p . && npm run build:nodebrowser && api-extractor run --local",
5455
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
5556
"clean": "rimraf dist dist-esm dist-test types *.tgz *.log dist-browser statistics.html coverage && rimraf src/**/*.js && rimraf test/**/*.js",
56-
"execute:js-samples": "echo skipped",
57-
"execute:ts-samples": "echo skipped",
58-
"execute:samples": "npm run build:samples && npm run execute:js-samples && npm run execute:ts-samples",
57+
"execute:samples": "npm run build:samples && echo skipped",
5958
"extract-api": "tsc -p . && api-extractor run --local",
6059
"format": "prettier --write --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
6160
"integration-test:browser": "echo skipped",

sdk/keyvault/keyvault-certificates/package.json

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,11 @@
4141
"audit": "node ../../../common/scripts/rush-audit.js && rimraf node_modules package-lock.json && npm i --package-lock-only 2>&1 && npm audit",
4242
"build:minify": "uglifyjs -c -m --comments --source-map \"content='./dist/index.js.map'\" -o ./dist/index.min.js ./dist/index.js 2>&1",
4343
"build:samples": "dev-tool samples prep && cd dist-samples && tsc",
44-
"build:es6": "tsc -p tsconfig.json",
44+
"build:node": "tsc -p . && cross-env ONLY_NODE=true rollup -c 2>&1",
45+
"build:browser": "tsc -p . && cross-env ONLY_BROWSER=true rollup -c 2>&1",
4546
"build:nodebrowser": "rollup -c 2>&1",
46-
"build:test": "npm run build:es6 && rollup -c rollup.test.config.js 2>&1",
47-
"build": "npm run extract-api && npm run build:es6 && npm run build:nodebrowser",
47+
"build:test": "tsc -p . && rollup -c rollup.test.config.js 2>&1",
48+
"build": "tsc -p . && npm run build:nodebrowser && api-extractor run --local",
4849
"check-format": "prettier --list-different --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
4950
"clean": "rimraf dist-esm dist-test typings *.tgz *.log samples/typescript/dist",
5051
"execute:samples": "npm run build:samples && dev-tool samples run dist-samples/javascript dist-samples/typescript/dist/keyvault-certificates/dist-samples/typescript/src/",

0 commit comments

Comments
 (0)