Merge pull request #586 from dropbigfish/develop

chore: fix some comments
protofire · Jun 19, 2024 · 13c6c11 · 13c6c11
2 parents cf6bf43 + 2b2a0a4
commit 13c6c11
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 7 deletions.
diff --git a/docs/rules/naming/named-parameters-mapping.md b/docs/rules/naming/named-parameters-mapping.md
@@ -39,7 +39,7 @@ mapping(string name => uint256 balance) public users;
 mapping(address owner => mapping(address token => uint256 balance)) public tokenBalances;
 ```
 
-#### Main key of mapping is enforced. On nested mappings other naming are not neccesary
+#### Main key of mapping is enforced. On nested mappings other naming are not necessary
 
 ```solidity
 mapping(address owner => mapping(address => uint256)) public tokenBalances;

diff --git a/e2e/test.js b/e2e/test.js
@@ -96,7 +96,7 @@ describe('e2e', function () {
     const PATH = '03-no-empty-blocks'
     const { PREFIX, SUFFIX } = prepareContext(PATH)
 
-    it('No contracts to lint should fail with appropiate message', function () {
+    it('No contracts to lint should fail with appropriate message', function () {
       const { code, stderr } = shell.exec(`${NODE}solhint Foo1.sol ${SUFFIX}`)
 
       expect(code).to.equal(EXIT_CODES.BAD_OPTIONS)

diff --git a/lib/rules/naming/func-named-parameters.js b/lib/rules/naming/func-named-parameters.js
@@ -71,7 +71,7 @@ class FunctionNamedParametersChecker extends BaseChecker {
       if (qtyNamed === 0 && qtyArgs > this.maxUnnamedArguments) {
         this.error(
           node,
-          `Named parameters missing. MIN unnamed argumenst is ${this.maxUnnamedArguments}`
+          `Named parameters missing. MIN unnamed arguments is ${this.maxUnnamedArguments}`
         )
       }
     }

diff --git a/lib/rules/naming/named-parameters-mapping.js b/lib/rules/naming/named-parameters-mapping.js
@@ -20,7 +20,7 @@ const meta = {
         },
         {
           description:
-            'Main key of mapping is enforced. On nested mappings other naming are not neccesary',
+            'Main key of mapping is enforced. On nested mappings other naming are not necessary',
           code: 'mapping(address owner => mapping(address => uint256)) public tokenBalances;',
         },
         {

diff --git a/test/rules/naming/func-named-parameters.js b/test/rules/naming/func-named-parameters.js
@@ -27,7 +27,7 @@ describe('Linter - func-named-parameters', () => {
       assertErrorCount(report, 1)
       assertErrorMessage(
         report,
-        `Named parameters missing. MIN unnamed argumenst is ${
+        `Named parameters missing. MIN unnamed arguments is ${
           minUnnamed < DEFAULT_MIN_UNNAMED_ARGUMENTS ? DEFAULT_MIN_UNNAMED_ARGUMENTS : minUnnamed
         }`
       )
@@ -90,7 +90,7 @@ describe('Linter - func-named-parameters', () => {
 
     assertErrorMessage(
       report,
-      `Named parameters missing. MIN unnamed argumenst is ${DEFAULT_MIN_UNNAMED_ARGUMENTS}`
+      `Named parameters missing. MIN unnamed arguments is ${DEFAULT_MIN_UNNAMED_ARGUMENTS}`
     )
   })
 
@@ -106,7 +106,7 @@ describe('Linter - func-named-parameters', () => {
 
     assertErrorMessage(
       report,
-      `Named parameters missing. MIN unnamed argumenst is ${DEFAULT_MIN_UNNAMED_ARGUMENTS}`
+      `Named parameters missing. MIN unnamed arguments is ${DEFAULT_MIN_UNNAMED_ARGUMENTS}`
     )
   })
 })