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 VRF V2, Fuzz Testing, NatSpec, Coverage #87

Merged
merged 4 commits into from
Mar 10, 2022
Merged

Conversation

andrejrakic
Copy link
Contributor

@andrejrakic andrejrakic commented Feb 28, 2022

Description

Related issues

Related issues are

@andrejrakic andrejrakic self-assigned this Feb 28, 2022
@andrejrakic andrejrakic mentioned this pull request Feb 28, 2022
test/unit/RandomNumberConsumerV2_unit_test.js Show resolved Hide resolved
deploy/03_Deploy_RandomNumberConsumer.js Outdated Show resolved Hide resolved
deploy/03_Deploy_RandomNumberConsumer.js Show resolved Hide resolved
hardhat.config.js Outdated Show resolved Hide resolved
@PatrickAlphaC
Copy link
Contributor

@andrejrakic all in all, this looks really good! Just a few comments. Wondering if it makes sense for us to fuzz more contracts? Probably not actually....

@PatrickAlphaC PatrickAlphaC merged commit 5492cde into main Mar 10, 2022
@andrejrakic andrejrakic deleted the chore/fix-issues branch March 17, 2022 11:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants