Skip to content

Commit 2e80939

Browse files
committed
restore full testing
1 parent fabce63 commit 2e80939

File tree

2 files changed

+7
-14
lines changed

2 files changed

+7
-14
lines changed

.github/workflows/test-all-versions.pr.yml

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,9 @@ jobs:
2626
OUTPUT=`node scripts/pr-labels-to-npm-workspace-args.mjs "$PR_LABELS"`
2727
echo "args=$OUTPUT" >> $GITHUB_OUTPUT
2828
29-
#XXX disable expensive tav run on this PR while in draft
30-
# tav:
31-
# uses: ./.github/workflows/test-all-versions.yml
32-
# needs: parse-labels
33-
# with:
34-
# npm-workspace-args: ${{ needs.parse-labels.outputs.args }}
35-
# if: ${{ needs.parse-labels.outputs.args != '' }}
29+
tav:
30+
uses: ./.github/workflows/test-all-versions.yml
31+
needs: parse-labels
32+
with:
33+
npm-workspace-args: ${{ needs.parse-labels.outputs.args }}
34+
if: ${{ needs.parse-labels.outputs.args != '' }}

.github/workflows/unit-test.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -170,13 +170,7 @@ jobs:
170170
run: npm run test
171171
- name: Unit tests (Delta)
172172
if: ${{ !matrix.code-coverage }}
173-
# XXX
174-
# run: npm run test:ci:changed
175-
# Getting a hang after testing instr-runtime-node on most Node.js versions,
176-
# or perhaps on the next test being run. Node.js v18 was the odd guy out
177-
# which was hanging after passing `baggage-span-processor` tests.
178-
# https://github.com/open-telemetry/opentelemetry-js-contrib/actions/runs/13715917502/job/38360869756?pr=2738
179-
run: npm run test:ci:changed -- --verbose --exclude @opentelemetry/baggage-span-processor,@opentelemetry/instrumentation-runtime-node
173+
run: npm run test:ci:changed
180174
- name: Report Coverage
181175
if: ${{ matrix.code-coverage && !cancelled()}}
182176
uses: codecov/codecov-action@v5

0 commit comments

Comments
 (0)