build(deps-dev): Bump solhint from 3.4.1 to 3.6.1

[dependabot]

Bumps solhint from 3.4.1 to 3.6.1.

Release notes

Sourced from solhint's releases.

v3.5.1

[3.5.1] - 2023-08-04

Updated

• Support ignoreConstructors option for no-empty-blocks #418
• Bump json5 from 2.1.3 to 2.2.3 #376
• Bump json-schema and jsprim #370
• Bump semver from 6.3.0 to 7.5.2 #438
• Corrected "Category" of quotes rule, added default rules list on readme #443
• 'Deprecated' column on `rules.md`` #444
• Information about maxCharacters allowed on reason-string rule #446
• E2E tests for max-warnings #455
• Replaced blacklist and whitelist words #459
• Removed runtime dependencies on load-rules #462
• Allowed $ symbol as part of naming #465
• Disabled no-empty-blocks rule for receive() function #466

Added

• New Rule: No unused imports #417
• New Rule: To treat immutable as constants #458
• New Rule: Explicit-types. To forbid/enforce full type or alias for variables declaration #467
• New Rule: Naming of Function parameters. Enforce arguments naming #468
• JSON formatter support #440
• Rules List with list-rules command #449
• E2E tests for formatters and new Compact formatter #457

Fixed

• maxWarnings parameter waiting review #439
• –fix option not working in avoid-throw rule #442
• Formatter option fixed for stdin command #450

Changelog

Sourced from solhint's changelog.

[3.6.1] - 2023-08-11

BREAKING CHANGE

• RULE: not-rely-on-time was REMOVED from RECOMMENDED ruleset This was long overdue. Beware!! If you are relying on this rule and it is not explicitly configured (meaning there's only solhint:recommended option). You should add this rule manually:

  {
    "extends": "solhint:recommended",
    "rules": { 
        "not-rely-on-time": "warn", 
        "compiler-version": "off" 
    },
  }

If not explicitly added, this rule will not be executed.

SPECIAL ATTENTION

• RULE: compiler-version default was updated from ^0.5.2 to ^0.8.0

Added

• New Rule: Enforces the use of Custom Errors over Require and Revert statements #475
• New Rule: Enforces the test_ prefix on a file for Foundry users #476
• New Rule: Enforces the naming of function return values #478
• Notes option on docs to add more information of each rule. See foundry-test-functions. #476

Fixed

• func-named-parameters - false positives on builtin functions #472
• ordering - treat initializer weight same as constructor #474
• check-send-result - false positive on `erc777.send()`` function #477
• explicit-types - default value is now taking into account when no value is specified in config #481
• compiler-version - default value is now taking into account when no value is specified in config #483

Updates

• Rule: check-send-result added config clarification in the new Notes section #482
• Rule: compiler-version default was updated from ^0.5.2 to ^0.8.0 #483

[3.5.1] - 2023-08-04

Updated

• Support ignoreConstructors option for no-empty-blocks #418
• Bump json5 from 2.1.3 to 2.2.3 #376
• Bump json-schema and jsprim #370
• Bump semver from 6.3.0 to 7.5.2 #438
• Corrected "Category" of quotes rule, added default rules list on readme #443
• 'Deprecated' column on `rules.md`` #444
• Information about maxCharacters allowed on reason-string rule #446
• E2E tests for max-warnings #455

... (truncated)

Commits

• 48258ca Merge pull request #473 from protofire/develop
• b67d523 Merge pull request #483 from protofire/i230-fix-compiler-version-default-rule
• 9228379 pre release fixes
• 84f366f Merge branch 'i230-fix-compiler-version-default-rule' of github.com:protofire...
• 75db884 fix and bump compiler-version rule default
• 14bb853 Merge branch 'develop' into i230-fix-compiler-version-default-rule
• 3ea57c5 fix and bump compiler version default
• 038e10d Merge pull request #482 from protofire/i295-update-check-send-result-docs
• 2876615 update-check-send-result-docs
• 34562bb Merge pull request #480 from protofire/no-unused-imports-fix
• Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

[dependabot]

Superseded by #456.