Skip to content

Commit

Permalink
chore: remove coveralls (#112)
Browse files Browse the repository at this point in the history
## Description

Coveralls is noisy, since it started adding a message to each PR since a
while ago.
We are also going to break full coverage while we migrate tests
gradually from Hardhat to Foundry.
This PR removes Coveralls from this repo.

## Test Plan

See no new message on CI.

## Related Issues

#106
  • Loading branch information
fedgiac authored Jun 4, 2024
1 parent 69443fe commit d644913
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 7 deletions.
6 changes: 0 additions & 6 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -30,9 +30,3 @@ jobs:
- run: yarn lint
- run: yarn coverage
- run: yarn test:ignored-in-coverage
- name: Coveralls
uses: coverallsapp/github-action@master
continue-on-error: true
with:
github-token: ${{ secrets.GITHUB_TOKEN }}

2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -109,7 +109,7 @@ npx hardhat tenderly:verify --network $NETWORK GPv2Contract=0xFeDbc87123caF39251

This package additionally contains a `networks.json` file at the root with the address of each deployed contract as well the hash of the Ethereum transaction used to create the contract.

## Test coverage [![Coverage Status](https://coveralls.io/repos/github/gnosis/gp-v2-contracts/badge.svg?branch=main)](https://coveralls.io/github/gnosis/gp-v2-contracts?branch=main)
## Test coverage

Test coverage can be checked with the command

Expand Down

0 comments on commit d644913

Please sign in to comment.