Skip to content

Commit

Permalink
test: drop separate helpers testing step
Browse files Browse the repository at this point in the history
Since minimum requirements are Node.js 18, helpers should be able to run together.
  • Loading branch information
ludofischer committed Apr 27, 2024
1 parent 78c6be5 commit 9b8018b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 22 deletions.
20 changes: 0 additions & 20 deletions .github/workflows/test.yml
Expand Up @@ -29,26 +29,6 @@ jobs:
- name: Run lint
run: pnpm lint

test_helpers:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4.1.3
- name: Install pnpm
uses: pnpm/action-setup@v3.0.0
with:
version: 9.0.6
- name: Install Node.js 20.x
uses: actions/setup-node@v4.0.2
with:
node-version: '20'
cache: 'pnpm'

- name: Install dependencies
run: pnpm install --frozen-lockfile

- name: Test helpers
run: pnpm test:helpers

test:
needs: lint
runs-on: ${{ matrix.os }}
Expand Down
3 changes: 1 addition & 2 deletions package.json
Expand Up @@ -6,10 +6,9 @@
"fixlint": "prettier --write . && pnpm lint -- --fix",
"build:integration": "node ./util/buildFrameworks.mjs",
"pretest": "pnpm lint",
"test:only": "node --test \"packages/**/test/**/*js\"",
"test:only": "node --test",
"test:coverage": "c8 pnpm test:only",
"test": "pnpm test:coverage",
"test:helpers": "node --test \"packages/**/test/**/*mjs\"",
"types": "tsc -b",
"all-publish": "pnpm changeset publish"
},
Expand Down

0 comments on commit 9b8018b

Please sign in to comment.