Skip to content

Commit

Permalink
[Docs] Adding docs npm script (#12941)
Browse files Browse the repository at this point in the history
This newly added command `docs` can help increase the quality of our documentation comments. It enables us to have a tight feedback loop on what is being generated as a documentation of our packages. I am pinning `typedoc` to v0.15.0 for now because this is the version being used for generating docs at `docs.microsoft.com`. This version should be updated when that team updates theirs.

Fixes #12928
  • Loading branch information
deyaaeldeen authored Dec 17, 2020
1 parent 5b0fe95 commit 7e0bd75
Show file tree
Hide file tree
Showing 57 changed files with 337 additions and 170 deletions.
190 changes: 126 additions & 64 deletions common/config/rush/pnpm-lock.yaml

Large diffs are not rendered by default.

6 changes: 4 additions & 2 deletions common/tools/dev-tool/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,8 @@
"pack": "npm pack 2>&1",
"prebuild": "npm run clean",
"unit-test": "mocha --require ts-node/register test/**/*.spec.ts",
"build:samples": "echo Skipped."
"build:samples": "echo Skipped.",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"repository": "github:Azure/azure-sdk-for-js",
"author": "Microsoft Corporation",
Expand Down Expand Up @@ -63,6 +64,7 @@
"rimraf": "^3.0.0",
"rollup": "^1.16.3",
"rollup-plugin-sourcemaps": "^0.4.2",
"rollup-plugin-visualizer": "^4.0.4"
"rollup-plugin-visualizer": "^4.0.4",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions common/tools/eslint-plugin-azure-sdk/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -51,7 +51,8 @@
"unit-test:node": "mocha --require source-map-support/register --timeout 10000 --full-trace --recursive dist/tests",
"unit-test:browser": "echo skipped",
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"test": "npm run clean && npm run build:test && npm run unit-test"
"test": "npm run clean && npm run build:test && npm run unit-test",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"engines": {
"node": ">=8.0.0"
Expand Down Expand Up @@ -89,6 +90,7 @@
"rimraf": "^3.0.0",
"source-map-support": "^0.5.9",
"mocha-junit-reporter": "^1.18.0",
"typescript": "4.1.2"
"typescript": "4.1.2",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/anomalydetector/ai-anomaly-detector/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@
"test": "npm run build:test && npm run unit-test && npm run integration-test",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 1200000 --full-trace dist-test/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -110,7 +111,8 @@
"rollup-plugin-visualizer": "^4.0.4",
"typescript": "4.1.2",
"util": "^0.12.1",
"@azure/test-utils-recorder": "^1.0.0"
"@azure/test-utils-recorder": "^1.0.0",
"typedoc": "0.15.0"
},
"//smokeTestConfiguration": {
"skipFolder": true
Expand Down
6 changes: 4 additions & 2 deletions sdk/appconfiguration/app-configuration/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,8 @@
"test": "npm run clean && npm run build:test && npm run unit-test",
"unit-test:browser": "",
"unit-test:node": "mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 180000 --full-trace test-dist/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"sideEffects": false,
"autoPublish": false,
Expand Down Expand Up @@ -133,6 +134,7 @@
"ts-node": "^8.3.0",
"typescript": "4.1.2",
"uglify-js": "^3.4.9",
"cross-env": "^7.0.2"
"cross-env": "^7.0.2",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/communication/communication-administration/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,8 @@
"test:watch": "npm run test -- --watch --reporter min",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "mocha --reporter ../../../common/tools/mocha-multi-reporter.js dist-test/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -119,6 +120,7 @@
"rollup-plugin-shim": "^1.0.0",
"rollup": "^1.16.3",
"sinon": "^9.0.2",
"typescript": "4.1.2"
"typescript": "4.1.2",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/communication/communication-chat/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,8 @@
"test": "npm run build:test && npm run unit-test && npm run integration-test",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "mocha --reporter ../../../common/tools/mocha-multi-reporter.js dist-test/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -121,6 +122,7 @@
"rollup": "^1.16.3",
"sinon": "^9.0.2",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/communication/communication-common/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@
"test": "npm run build:test && npm run unit-test && npm run integration-test",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "mocha --reporter ../../../common/tools/mocha-multi-reporter.js dist-test/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -112,6 +113,7 @@
"rollup": "^1.16.3",
"sinon": "^9.0.2",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/communication/communication-sms/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,8 @@
"test": "npm run build:test && npm run unit-test && npm run integration-test",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "mocha --reporter ../../../common/tools/mocha-multi-reporter.js dist-test/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -119,7 +120,8 @@
"rollup": "^1.16.3",
"sinon": "^9.0.2",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
},
"//smokeTestConfiguration": {
"skipFolder": true
Expand Down
6 changes: 4 additions & 2 deletions sdk/core/abort-controller/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,8 @@
"unit-test:browser": "karma start --single-run",
"unit-test:node": "cross-env TS_NODE_FILES=true TS_NODE_COMPILER_OPTIONS=\"{\\\"module\\\": \\\"commonjs\\\"}\" mocha --require ts-node/register --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --full-trace --no-timeouts test/*.spec.ts",
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"build:samples": "echo Skipped."
"build:samples": "echo Skipped.",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"types": "./types/src/index.d.ts",
"engine": {
Expand Down Expand Up @@ -102,6 +103,7 @@
"rollup-plugin-sourcemaps": "^0.4.2",
"rollup-plugin-terser": "^5.1.1",
"ts-node": "^8.3.0",
"typescript": "4.1.2"
"typescript": "4.1.2",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-amqp/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,8 @@
"test": "npm run test:node && npm run test:browser",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "cross-env TS_NODE_FILES=true TS_NODE_COMPILER_OPTIONS=\"{\\\"module\\\":\\\"commonjs\\\"}\" nyc --reporter=lcov --reporter=text-lcov mocha -r ts-node/register -t 50000 --reporter ../../../common/tools/mocha-multi-reporter.js ./test/**/*.spec.ts",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"engines": {
"node": ">=8.0.0"
Expand Down Expand Up @@ -124,6 +125,7 @@
"sinon": "^9.0.2",
"ts-node": "^8.3.0",
"typescript": "4.1.2",
"ws": "^7.1.1"
"ws": "^7.1.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-asynciterator-polyfill/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,8 @@
"unit-test:browser": "echo skipped",
"unit-test:node": "echo skipped",
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"build:samples": "echo Skipped."
"build:samples": "echo Skipped.",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"sideEffects": true,
"private": false,
Expand All @@ -59,6 +60,7 @@
"@types/node": "^8.0.0",
"eslint": "^7.15.0",
"prettier": "^1.16.4",
"typescript": "4.1.2"
"typescript": "4.1.2",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-auth/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -38,7 +38,8 @@
"test": "npm run build:test && npm run unit-test && npm run integration-test",
"unit-test:browser": "echo skipped",
"unit-test:node": "mocha test-dist/**/*.js --reporter ../../../common/tools/mocha-multi-reporter.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -94,6 +95,7 @@
"rollup-plugin-terser": "^5.1.1",
"rollup-plugin-visualizer": "^4.0.4",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-client/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,8 @@
"test": "npm run clean && npm run build:ts && npm run bundle:test:node && npm run unit-test:node && npm run bundle:test:browser && npm run unit-test:browser && npm run integration-test:node && npm run integration-test:browser",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "cross-env TS_NODE_FILES=true mocha -r esm -r ts-node/register --timeout 50000 --reporter ../../../common/tools/mocha-multi-reporter.js --colors --exclude \"test/**/*.browser.ts\" \"test/**/*.ts\"",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -121,6 +122,7 @@
"rollup-plugin-visualizer": "^4.0.4",
"sinon": "^9.0.2",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-http/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -101,7 +101,8 @@
"publish-preview": "mocha --no-colors && shx rm -rf dist/test && node ./.scripts/publish",
"local": "ts-node ./.scripts/local.ts",
"latest": "ts-node ./.scripts/latest.ts",
"build:samples": "echo Skipped."
"build:samples": "echo Skipped.",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"sideEffects": false,
"nyc": {
Expand Down Expand Up @@ -194,6 +195,7 @@
"ts-node": "^8.3.0",
"typescript": "4.1.2",
"uglify-js": "^3.4.9",
"xhr-mock": "^2.4.1"
"xhr-mock": "^2.4.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-https/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,8 @@
"test": "npm run clean && npm run build:ts && npm run bundle:test:node && npm run unit-test:node && npm run bundle:test:browser && npm run unit-test:browser && npm run integration-test:node && npm run integration-test:browser",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js dist-test/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -137,6 +138,7 @@
"sinon": "^9.0.2",
"source-map-support": "^0.5.9",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-lro/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -87,7 +87,8 @@
"unit-test:browser": "npm run build:test && karma start --single-run",
"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",
"pack": "npm pack 2>&1",
"prebuild": "npm run clean"
"prebuild": "npm run clean",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"sideEffects": false,
"dependencies": {
Expand Down Expand Up @@ -134,6 +135,7 @@
"rollup-plugin-visualizer": "^4.0.4",
"ts-node": "^8.3.0",
"typescript": "4.1.2",
"uglify-js": "^3.4.9"
"uglify-js": "^3.4.9",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-paging/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,8 @@
"unit-test:browser": "echo skipped",
"unit-test:node": "echo skipped",
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"build:samples": "echo Skipped."
"build:samples": "echo Skipped.",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"sideEffects": true,
"private": false,
Expand All @@ -75,6 +76,7 @@
"eslint": "^7.15.0",
"prettier": "^1.16.4",
"rimraf": "^3.0.0",
"typescript": "4.1.2"
"typescript": "4.1.2",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-tracing/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,8 @@
"test": "npm run build:test && npm run unit-test && npm run integration-test",
"unit-test:browser": "echo skipped",
"unit-test:node": "mocha test-dist/**/*.js --reporter ../../../common/tools/mocha-multi-reporter.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -86,6 +87,7 @@
"rollup-plugin-terser": "^5.1.1",
"rollup-plugin-visualizer": "^4.0.4",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/core-xml/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,8 @@
"test": "npm run clean && npm run build:ts && npm run bundle:test:node && npm run unit-test:node && npm run bundle:test:browser && npm run unit-test:browser && npm run integration-test:node && npm run integration-test:browser",
"unit-test:browser": "karma start --single-run",
"unit-test:node": "mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js dist-test/index.node.js",
"unit-test": "npm run unit-test:node && npm run unit-test:browser"
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"files": [
"dist/",
Expand Down Expand Up @@ -117,6 +118,7 @@
"rollup-plugin-visualizer": "^4.0.4",
"sinon": "^9.0.2",
"typescript": "4.1.2",
"util": "^0.12.1"
"util": "^0.12.1",
"typedoc": "0.15.0"
}
}
6 changes: 4 additions & 2 deletions sdk/core/logger/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,8 @@
"unit-test:browser": "karma start --single-run",
"unit-test:node": "cross-env TS_NODE_FILES=true TS_NODE_COMPILER_OPTIONS=\"{\\\"module\\\": \\\"commonjs\\\"}\" mocha --require ts-node/register --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --full-trace --no-timeouts test/*.spec.ts",
"unit-test": "npm run unit-test:node && npm run unit-test:browser",
"build:samples": "echo Skipped."
"build:samples": "echo Skipped.",
"docs": "typedoc --excludePrivate --excludeNotExported --mode file --out ./dist/docs ./src"
},
"types": "./types/logger.d.ts",
"engine": {
Expand Down Expand Up @@ -111,6 +112,7 @@
"rollup-plugin-terser": "^5.1.1",
"sinon": "^9.0.2",
"ts-node": "^8.3.0",
"typescript": "4.1.2"
"typescript": "4.1.2",
"typedoc": "0.15.0"
}
}
2 changes: 1 addition & 1 deletion sdk/cosmosdb/cosmos/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@
"ts-node": "^8.3.0",
"tslint": "^5.0.0",
"tslint-config-prettier": "^1.14.0",
"typedoc": "^0.15.0",
"typedoc": "0.15.0",
"typescript": "4.1.2"
}
}
Loading

0 comments on commit 7e0bd75

Please sign in to comment.