Skip to content

Commit

Permalink
chore: fix issue with nyc crashing (#1605)
Browse files Browse the repository at this point in the history
* chore: fix issue with nyc crashing

* chore: revert hoist change, make @babel/core nohoist

* chore: update @babel/core to test Garbage Collector issue

* chore: bump typescripts to 4.4.4 in some package

* Revert "chore: bump typescripts to 4.4.4 in some package"

This reverts commit 5bf1bfb.

* Revert "Revert "chore: bump typescripts to 4.4.4 in some package""

This reverts commit 2877b7e.

* Fix cancelled issue

---------

Co-authored-by: Haddas Bronfman <85441461+haddasbronfman@users.noreply.github.com>
  • Loading branch information
chigia001 and haddasbronfman authored Aug 10, 2023
1 parent 154b30b commit a1243ee
Show file tree
Hide file tree
Showing 6 changed files with 14 additions and 7 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/unit-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,9 @@ jobs:
fail-fast: false
matrix:
node: ["14", "16", "18"]
include:
- node: 14
code-coverage: true
runs-on: ubuntu-latest
services:
memcached:
Expand Down Expand Up @@ -146,12 +149,16 @@ jobs:
run: npm install --ignore-scripts
- name: Bootstrap Dependencies
run: npx lerna bootstrap --no-ci --hoist --nohoist='zone.js' --nohoist='mocha' --nohoist='ts-mocha'
- name: Unit tests
- name: Unit tests (Full)
if: matrix.code-coverage
run: npm run test -- ${{ matrix.lerna-extra-args }}
- name: Unit tests (Delta)
if: ${{ !matrix.code-coverage }}
run: npm run test:ci:changed -- ${{ matrix.lerna-extra-args }}
- name: Build examples
run: npm run compile:examples
- name: Report Coverage
if: matrix.node == '14'
if: ${{ matrix.code-coverage && !cancelled()}}
uses: codecov/codecov-action@v3
with:
verbose: true
Expand Down
2 changes: 1 addition & 1 deletion metapackages/auto-instrumentations-web/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@
"@opentelemetry/api": "^1.3.0"
},
"devDependencies": {
"@babel/core": "7.15.0",
"@babel/core": "7.22.9",
"@opentelemetry/api": "^1.3.0",
"@types/mocha": "8.2.3",
"@types/node": "18.16.19",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,7 @@
"@opentelemetry/api": "^1.3.0"
},
"devDependencies": {
"@babel/core": "7.15.0",
"@babel/core": "7.22.9",
"@opentelemetry/api": "^1.3.0",
"@types/mocha": "8.2.3",
"@types/node": "18.16.19",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
"access": "public"
},
"devDependencies": {
"@babel/core": "7.15.0",
"@babel/core": "7.22.9",
"@opentelemetry/api": "^1.3.0",
"@opentelemetry/sdk-trace-base": "^1.8.0",
"@types/jquery": "3.5.16",
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
"access": "public"
},
"devDependencies": {
"@babel/core": "7.15.0",
"@babel/core": "7.22.9",
"@opentelemetry/api": "^1.3.0",
"@opentelemetry/context-zone-peer-dep": "^1.8.0",
"@opentelemetry/instrumentation-xml-http-request": "^0.41.0",
Expand Down
2 changes: 1 addition & 1 deletion plugins/web/opentelemetry-plugin-react-load/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@
"access": "public"
},
"devDependencies": {
"@babel/core": "7.15.0",
"@babel/core": "7.22.9",
"@opentelemetry/api": "^1.0.0",
"@opentelemetry/propagator-b3": "^1.3.1",
"@types/mocha": "7.0.2",
Expand Down

0 comments on commit a1243ee

Please sign in to comment.