Skip to content

Commit

Permalink
chore: publish /src to npm and other fixes for sourcemap debugging (#…
Browse files Browse the repository at this point in the history 
…1462)

* fix: remove .npmignore's /src/ (for sourcemaps)

Crrently, SDKV3 publishes sourcemaps to npm but doesn't publish source
files that those sourcemaps point to. This missing source files breaks
debugging use-cases, especially for VSCode users because the VSCode
debugger relies on source files for setting breakpoints in the debugger,
for debugger call stacks, for "step into" original source, and any other
debugger use-cases. Even outside of debugging use-cases, it's helpful
for developers consuming transpiled libraries to have original source so
they can better understand what the library is doing. Finally, some tools
will show warnings when sourcemaps are present but source files are
missing, and this commit will fix these warnings.

* fix: add jest.config.js to .npmignore

* fix: add *.spec.ts to nonclient package .npmignore

* fix: change sourceRoot->rootDir in tsconfig.*.json
  • Loading branch information
@justingrant
justingrant committed Oct 7, 2020
1 parent 47c6599 commit f57c709
Show file tree
Hide file tree
Showing 98 changed files with 180 additions and 98 deletions.
3 changes: 2 additions & 1 deletion packages/abort-controller/.npmignore
View file
@@ -1,9 +1,10 @@
/src/
/coverage/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/body-checksum-browser/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/body-checksum-node/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/chunked-blob-reader-native/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/chunked-blob-reader/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/chunked-stream-reader-node/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/client-documentation-generator/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/config-resolver/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/credential-provider-cognito-identity/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/credential-provider-env/.npmignore
View file
@@ -1,9 +1,10 @@
/src/
/coverage/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-env/tsconfig.cjs.json
View file
Expand Up @@ -4,7 +4,7 @@
"strict": true,
"sourceMap": false,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/cjs",
"importHelpers": true,
"noEmitHelpers": true,
Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-env/tsconfig.es.json
View file
Expand Up @@ -4,7 +4,7 @@
"strict": true,
"sourceMap": false,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/es",
"importHelpers": true,
"noEmitHelpers": true,
Expand Down
3 changes: 2 additions & 1 deletion packages/credential-provider-imds/.npmignore
View file
@@ -1,9 +1,10 @@
/src/
/coverage/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-imds/tsconfig.cjs.json
View file
Expand Up @@ -5,7 +5,7 @@
"sourceMap": false,
"importHelpers": true,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/cjs",
"noEmitHelpers": true,
"inlineSourceMap": true,
Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-imds/tsconfig.es.json
View file
Expand Up @@ -5,7 +5,7 @@
"sourceMap": false,
"importHelpers": true,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/es",
"noEmitHelpers": true,
"inlineSourceMap": true,
Expand Down
3 changes: 2 additions & 1 deletion packages/credential-provider-ini/.npmignore
View file
@@ -1,9 +1,10 @@
/src/
/coverage/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-ini/tsconfig.cjs.json
View file
Expand Up @@ -5,7 +5,7 @@
"sourceMap": false,
"importHelpers": true,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/cjs",
"noEmitHelpers": true,
"inlineSourceMap": true,
Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-ini/tsconfig.es.json
View file
Expand Up @@ -5,7 +5,7 @@
"sourceMap": false,
"importHelpers": true,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/es",
"noEmitHelpers": true,
"inlineSourceMap": true,
Expand Down
3 changes: 2 additions & 1 deletion packages/credential-provider-node/.npmignore
View file
@@ -1,9 +1,10 @@
/src/
/coverage/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/credential-provider-process/.npmignore
View file
@@ -1,9 +1,10 @@
/src/
/coverage/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-process/tsconfig.cjs.json
View file
Expand Up @@ -5,7 +5,7 @@
"sourceMap": false,
"importHelpers": true,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/cjs",
"noEmitHelpers": true,
"inlineSourceMap": true,
Expand Down
2 changes: 1 addition & 1 deletion packages/credential-provider-process/tsconfig.es.json
View file
Expand Up @@ -5,7 +5,7 @@
"sourceMap": false,
"importHelpers": true,
"lib": ["es5", "es2015.promise", "es2015.collection"],
"sourceRoot": "./src",
"rootDir": "./src",
"outDir": "./dist/es",
"noEmitHelpers": true,
"inlineSourceMap": true,
Expand Down
3 changes: 2 additions & 1 deletion packages/eventstream-handler-node/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/eventstream-marshaller/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/eventstream-serde-browser/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/eventstream-serde-config-resolver/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/eventstream-serde-node/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/eventstream-serde-universal/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/fetch-http-handler/.npmignore
View file
@@ -1,9 +1,10 @@
/src/
/coverage/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/hash-blob-browser/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/hash-node/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/hash-stream-node/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down
3 changes: 2 additions & 1 deletion packages/invalid-dependency/.npmignore
View file
@@ -1,10 +1,11 @@
/src/
/coverage/
/docs/
tsconfig.test.json
*.tsbuildinfo
jest.config.js

*.spec.js
*.spec.ts
*.spec.d.ts
*.spec.js.map

Expand Down

0 comments on commit f57c709

Please sign in to comment.