NoBey/openzeppelin-contracts
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Add override guidelines

Browse Source
This commit is contained in:
Francisco
2023-07-03 12:05:50 -03:00
committed by GitHub
parent 06861dce54
commit ac48658397
1 changed files with 5 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
GUIDELINES.md
View File

@ -136,3 +136,8 @@ In addition to the official Solidity Style Guide we have a number of other conve
4. Declare the error in an extension if the error only happens in such extension or child contracts. 4. Declare the error in an extension if the error only happens in such extension or child contracts.
* Custom error names should not be declared twice along the library to avoid duplicated identifier declarations when inheriting from multiple contracts. * Custom error names should not be declared twice along the library to avoid duplicated identifier declarations when inheriting from multiple contracts.
* It is important to follow a few rules around the use of Solidity overrides to avoid introducing unintended consequences due to the interaction with multiple inheritance:
1. When overriding a function `foo`, always invoke `super.foo`, and pass the same arguments that were received.
2. Never use `super` for a function other than the one being overridden.