Skip to content

Commit

Permalink
'No uppercase' warning for public vars removed
Browse files Browse the repository at this point in the history
  • Loading branch information
bart1e committed Dec 22, 2022
1 parent 7ea5727 commit 3db1e32
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion slither/detectors/naming_convention/naming_convention.py
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,8 @@ def _detect(self): # pylint: disable=too-many-branches,too-many-statements
# For ERC20 compatibility
if var.name in ["symbol", "name", "decimals"]:
continue

if var.visibility == "public":
continue
if not self.is_upper_case_with_underscores(var.name):
info = [
"Constant ",
Expand Down

0 comments on commit 3db1e32

Please sign in to comment.