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

Remove unnecessary virtual #3650

Merged
merged 5 commits into from
Aug 31, 2022
Merged

Remove unnecessary virtual #3650

merged 5 commits into from
Aug 31, 2022

Conversation

fulldecent
Copy link
Contributor

@fulldecent fulldecent commented Aug 26, 2022

This PR removes unnecessary virtual in all mock contracts where those mocks are not intended to be subclassed.

This reduces unnecessary surface area of those mocks and also reduces the risk of somebody seeing those mocks and thinking they are intended to be subclassed.

This was referenced Aug 26, 2022
@frangio
Copy link
Contributor

frangio commented Aug 31, 2022

Thank you @fulldecent. Our automated changelog check is a little simple and this was a false positive since it's not a user facing change. Can you remove the changelog entry?

@fulldecent
Copy link
Contributor Author

@frangio cool, all done, other checks passed

Copy link
Contributor

@frangio frangio left a comment

Choose a reason for hiding this comment

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

Thanks!

@frangio frangio merged commit 1eb55e2 into OpenZeppelin:master Aug 31, 2022
@gitpoap-bot
Copy link

gitpoap-bot bot commented Aug 31, 2022

Woohoo, your important contribution to this open-source project has earned you a GitPOAP!

GitPOAP: 2022 OpenZeppelin Contracts Contributor:
GitPOAP: 2022 OpenZeppelin Contracts Contributor GitPOAP Badge

Head on over to GitPOAP.io and connect your GitHub account to mint!

@fulldecent fulldecent deleted the patch-14 branch September 1, 2022 13:51
ronhuafeng added a commit to ronhuafeng/openzeppelin-contracts that referenced this pull request Sep 9, 2022
JulissaDantes pushed a commit to JulissaDantes/openzeppelin-contracts that referenced this pull request Nov 3, 2022
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

Successfully merging this pull request may close these issues.

2 participants