Skip to content

Commit

Permalink
⚡ chore(ci): Remove pnpm from test coverage action (#576)
Browse files Browse the repository at this point in the history
## Description

remove pnpm from test coverage ci action to speed up test action from 2
minutes to ___

Test coverage worked great locally but fails and misses test coverage in
ci

Closes #577

## Testing

Explain the quality checks that have been done on the code changes

## Additional Information

- [ ] I read the [contributing docs](../docs/contributing.md) (if this
is your first contribution)

Your ENS/address:

Co-authored-by: Will Cory <[email protected]>
  • Loading branch information
roninjin10 and Will Cory committed Oct 24, 2023
1 parent cb7555a commit a28ad33
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 67 deletions.
41 changes: 0 additions & 41 deletions .github/actions/setup-pnpm/action.yml

This file was deleted.

4 changes: 2 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,10 @@ jobs:
submodules: recursive

- name: "Setup"
uses: ./.github/actions/setup-pnpm
uses: ./.github/actions/setup

- name: Run tests
run: pnpm nx run-many --target=test:coverage
run: bun nx run-many --target=test:coverage
env:
NX_CLOUD_ACCESS_TOKEN: ${{ secrets.NX_CLOUD_ACCESS_TOKEN }}
RPC_URL_MAINNET: ${{ secrets.RPC_URL_MAINNET }}
Expand Down
24 changes: 0 additions & 24 deletions pnpm-workspace.yaml

This file was deleted.

1 comment on commit a28ad33

@vercel
Copy link

@vercel vercel bot commented on a28ad33 Oct 24, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Successfully deployed to the following URLs:

evmts-docs – ./

evmts-docs-git-main-evmts.vercel.app
evmts.dev
evmts-docs-evmts.vercel.app

Please sign in to comment.