-
Notifications
You must be signed in to change notification settings - Fork 12.4k
Remove override interface implementations #4315
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
Merged
ernestognw
merged 31 commits into
OpenZeppelin:master
from
RenanSouza2:remove-override-interface-implementations
Jun 7, 2023
Merged
Changes from 1 commit
Commits
Show all changes
31 commits
Select commit
Hold shift + click to select a range
4daab31
AccessControl
RenanSouza2 ebf635c
AccessControlEnumerable
RenanSouza2 7a0466f
Governor
RenanSouza2 f377dd4
IGovernor
RenanSouza2 6b50b81
TimelockController
RenanSouza2 bec26fc
Votes
RenanSouza2 17401ed
ERC1271WalletMock
RenanSouza2 8db131c
ERC3156FlashBorrowerMock
RenanSouza2 5d29695
ERC20VotesLegacyMock
RenanSouza2 75cb14f
ERC721ReceiverMock
RenanSouza2 92dad67
ERC1155ReceiverMock
RenanSouza2 f20a8d1
UpgradeableBeacon
RenanSouza2 86519c3
UUPSUpgradeable
RenanSouza2 88e9853
ERC2981
RenanSouza2 cdf8b1d
ERC1155
RenanSouza2 4326964
ERC20
RenanSouza2 320a0c2
ERC20FlashMint
RenanSouza2 5da5221
ERC20Permit
RenanSouza2 9fb85db
ERC4626
RenanSouza2 8cc7431
ERC721
RenanSouza2 ff241a8
ERC721Enumerable
RenanSouza2 da12258
ERC721Wrapper
RenanSouza2 6ee337d
ERC721Holder
RenanSouza2 3cdda53
EIP712
RenanSouza2 307c154
EIP165
RenanSouza2 1372aeb
Recover comments
RenanSouza2 ac0a4b2
Add changeset
RenanSouza2 13b211b
Fix changeset
RenanSouza2 e6d8ee1
Update .changeset/violet-dancers-cough.md
ernestognw 52229bf
Merge remote-tracking branch 'upstream/master' into remove-override-i…
RenanSouza2 6ab19df
Merge branch 'master' into remove-override-interface-implementations
RenanSouza2 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Add changeset
- Loading branch information
commit ac0a4b2d9f271e997dec0d4427112c91c4d7f17b
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| --- | ||
| 'openzeppelin-solidity': patch | ||
| --- | ||
|
|
||
| Remove override modifier when not needed | ||
ernestognw marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.