single word constant variables trigger snake case error #11

protofire · Nov 17, 2017 · 125596a · 125596a
1 parent ea4a01f
commit 125596a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/README.md b/README.md
@@ -145,7 +145,7 @@ https://consensys.github.io/smart-contract-best-practices/recommendations/)
  | **func-param-name-mixedcase** | Function param name must be in mixedCase                                  |
  | **var-name-mixedcase**        | Variable name must be in mixedCase                                        |
  | **event-name-camelcase**      | Event name must be in CamelCase                                           |
- | **const-name-snakecase**      | Constant name must be in SCREAMING_SNAKE_CASE                             |
+ | **const-name-snakecase**      | Constant name must be in capitalized SNAKE_CASE                           |
  | **modifier-name-mixedcase**   | Modifier name must be in mixedCase                                        |
  | **contract-name-camelcase**   | Contract name must be in CamelCase                                        |
  | **use-forbidden-name**        | Avoid to use letters 'I', 'l', 'O' as identifiers                         |

diff --git a/docs/rules.md b/docs/rules.md
@@ -51,7 +51,7 @@ description: List of validation rules for Solhint - Solidity security, style gui
  | **func-param-name-mixedcase** | Function param name must be in mixedCase           | *[default](#options)*          |
  | **var-name-mixedcase**        | Variable name must be in mixedCase                 | *[default](#options)*          |
  | **event-name-camelcase**      | Event name must be in CamelCase                    | *[default](#options)*          |
- | **const-name-snakecase**      | Constant name must be in SCREAMING_SNAKE_CASE      | *[default](#options)*          |
+ | **const-name-snakecase**      | Constant name must be in capitalized SNAKE_CASE    | *[default](#options)*          |
  | **modifier-name-mixedcase**   | Modifier name must be in mixedCase                 | *[default](#options)*          |
  | **contract-name-camelcase**   | Contract name must be in CamelCase                 | *[default](#options)*          |
  | **use-forbidden-name**        | Avoid to use letters 'I', 'l', 'O' as identifiers  | *[default](#options)*          |

diff --git a/lib/rules/naming/var-name-style.js b/lib/rules/naming/var-name-style.js
@@ -43,7 +43,7 @@ class VarNameStyleChecker extends BaseChecker {
     validateConstantName(ctx) {
         this._forEachIdentifier(ctx, (curId, text) => {
             if (naming.isNotUpperSnakeCase(text)) {
-                this.error(curId, 'const-name-snakecase', 'Constant name must be in SCREAMING_SNAKE_CASE');
+                this.error(curId, 'const-name-snakecase', 'Constant name must be in capitalized SNAKE_CASE');
             }
         });
     }