Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add solidity-coverage #1

Open
chatch opened this issue Apr 11, 2019 · 2 comments
Open

add solidity-coverage #1

chatch opened this issue Apr 11, 2019 · 2 comments

Comments

@chatch
Copy link
Member

chatch commented Apr 11, 2019

Add solidity-coverage build step to evaluate tests coverage.

@chatch
Copy link
Member Author

chatch commented Apr 11, 2019

currently hit this issue on the calldata keyword on Feeds.addMarket: sc-forks/solidity-coverage#316 (comment)

Either wait for the fix or build and use the mentioned fork that fixes this.

@chatch
Copy link
Member Author

chatch commented Jul 12, 2019

above issue fixed in latest release: https://github.com/sc-forks/solidity-coverage/releases/tag/v0.6.0

give it a try.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant