From d206a254a7fd9fa8dff2879f3359a6468a740866 Mon Sep 17 00:00:00 2001 From: Vasco Santos Date: Wed, 14 Apr 2021 14:57:16 +0200 Subject: [PATCH] chore: apply suggestions from code review Co-authored-by: Alex Potsides --- .github/workflows/main.yml | 6 +++--- test/core/ping.node.js | 3 ++- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml index 3d9b5969e4..50b74aad04 100644 --- a/.github/workflows/main.yml +++ b/.github/workflows/main.yml @@ -14,7 +14,7 @@ jobs: - uses: actions/checkout@v2 - run: npm install - run: npx aegir lint - - run: npx aegir ts -p check + - uses: gozala/typescript-error-reporter-action@v1.0.8 - run: npx aegir build - run: npx aegir dep-check - uses: ipfs/aegir/actions/bundle-size@master @@ -35,7 +35,7 @@ jobs: with: node-version: ${{ matrix.node }} - run: npm install - - run: npx nyc --reporter=lcov aegir test -t node -- --bail + - run: npx aegir test -t node --cov --bail - uses: codecov/codecov-action@v1 test-chrome: needs: check @@ -50,7 +50,7 @@ jobs: steps: - uses: actions/checkout@v2 - run: npm install - - run: npx aegir test -t browser -t webworker --bail -- --browsers FirefoxHeadless + - run: npx aegir test -t browser -t webworker --bail -- --browser firefox test-ts: needs: check runs-on: ubuntu-latest diff --git a/test/core/ping.node.js b/test/core/ping.node.js index 30c350b6d8..4bbd4f76cc 100644 --- a/test/core/ping.node.js +++ b/test/core/ping.node.js @@ -58,7 +58,8 @@ describe('ping', () => { if (firstInvocation) { firstInvocation = false - for await (const data of stream) { // eslint-disable-line + // eslint-disable-next-line no-unreachable-loop + for await (const data of stream) { return { value: data, done: false