Skip to content

Commit ac36598

Browse files
willarmirostrivikr
andauthored
Re-add AWS V3 Instrumentation (#421)
* Revert "Revert PR #386 (#412)" This reverts commit 9e20d2e. * Add proper TypeScript build logic (#417) * build core files to dist directory * fixed unit tests * updated workflows and readme * try to fix windows * fix ls * slashes * no more ls * zstd decompress * see what bin has * removed rsync * fixes * added sh * try xargs instead * cleanups * fixed publishing logic * Redesign AWS SDK V3 instrumentation to use middleware (#416) * updated deps and type file * removed type keyword added deps * remove changes to js * updated versions * finished redesign of aws sdk v3 instrumentation * refactored buildAttributes signature * add compile back to workflow * bumped tsd version instead Co-authored-by: Trivikram Kamat <[email protected]> * removed dependencies and added docs * add back most changes, update types * removed aws-v3 branch from CI * fixed codecov to use dist directory * added blog post link Co-authored-by: Trivikram Kamat <[email protected]>
1 parent b616b51 commit ac36598

29 files changed

+893
-246
lines changed

.github/workflows/pr-build.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ jobs:
4444
if: '!matrix.coverage'
4545
run: |
4646
npx lerna bootstrap --hoist
47+
npx lerna run compile
4748
npx lerna run test
4849
shell: bash
4950
env:

.github/workflows/release.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,7 @@ jobs:
3131
- name: Execute tests with Lerna
3232
run: |
3333
npx lerna bootstrap --hoist
34+
npx lerna run compile
3435
npx lerna run test
3536
3637
- name: Publish package to npm

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,3 +4,4 @@ docs
44
.idea/
55
.nyc_output/
66
*.lcov
7+
dist

.nycrc.json

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,14 @@
11
{
22
"all": true,
3+
"include": [
4+
"**/dist/lib/**/*.js"
5+
],
36
"exclude": [
4-
"**/Gruntfile.js",
5-
"**/.prettierrc.js",
67
"**/*.d.ts",
78
"**/sample/**",
89
"**/test/**",
910
"**/test-d/**",
10-
"**/test_async/**"
11+
"**/test_async/**",
12+
"**/docs/**"
1113
]
1214
}

README.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,14 +89,18 @@ AWS will not:
8989

9090
## Testing from Source
9191

92-
This repo uses [Lerna](https://lernajs.io) to manage multiple packages. To install Lerna:
92+
This repo uses [Lerna](https://lernajs.io) to manage multiple packages. To install Lerna as a CLI:
9393
```
94-
npm install lerna
94+
npm install -g lerna
9595
```
9696
To install devDependencies and peerDependencies for all packages:
9797
```
9898
lerna bootstrap --hoist
9999
```
100+
This repo has a combination of TypeScript and JavaScript source files. To transpile the TypeScript files for testing, run:
101+
```
102+
lerna run compile
103+
```
100104
To run tests for all packages:
101105
```
102106
lerna run test

0 commit comments

Comments
 (0)