Merge pull request #355 from MCarlomagno/dev

fix grammar for time-based decisions
protofire · Dec 12, 2022 · abbdb02 · abbdb02
2 parents a7da9ce + 75d90da
commit abbdb02
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/docs/rules.md b/docs/rules.md
@@ -64,7 +64,7 @@ title:       "Rule Index of Solhint"
 | [no-complex-fallback](./rules/security/no-complex-fallback.md)         | Fallback function must be simple.                                        | ✔️          |
 | [no-inline-assembly](./rules/security/no-inline-assembly.md)           | Avoid to use inline assembly. It is acceptable only in rare cases.       | ✔️          |
 | [not-rely-on-block-hash](./rules/security/not-rely-on-block-hash.md)   | Do not rely on "block.blockhash". Miners can influence its value.        | ✔️          |
-| [not-rely-on-time](./rules/security/not-rely-on-time.md)               | Avoid to make time-based decisions in your business logic.               | ✔️          |
+| [not-rely-on-time](./rules/security/not-rely-on-time.md)               | Avoid making time-based decisions in your business logic.               | ✔️          |
 | [reentrancy](./rules/security/reentrancy.md)                           | Possible reentrancy vulnerabilities. Avoid state changes after transfer. | ✔️          |
 | [state-visibility](./rules/security/state-visibility.md)               | Explicitly mark visibility of state.                                     | ✔️          |
         

diff --git a/docs/rules/security/not-rely-on-time.md b/docs/rules/security/not-rely-on-time.md
@@ -12,7 +12,7 @@ title:       "not-rely-on-time | Solhint"
 
 
 ## Description
-Avoid to make time-based decisions in your business logic.
+Avoid making time-based decisions in your business logic.
 
 ## Options
 This rule accepts a string option of rule severity. Must be one of "error", "warn", "off". Default to warn.

diff --git a/lib/rules/security/not-rely-on-time.js b/lib/rules/security/not-rely-on-time.js
@@ -5,7 +5,7 @@ const meta = {
   type: 'security',
 
   docs: {
-    description: `Avoid to make time-based decisions in your business logic.`,
+    description: `Avoid making time-based decisions in your business logic.`,
     category: 'Security Rules'
   },
 
@@ -34,7 +34,7 @@ class NotRelyOnTimeChecker extends BaseChecker {
   }
 
   _warn(node) {
-    this.warn(node, 'Avoid to make time-based decisions in your business logic')
+    this.warn(node, 'Avoid making time-based decisions in your business logic')
   }
 }