Skip to content

chore(deps): update dependency solhint to v6#1469

Open
renovate[bot] wants to merge 1 commit intomainfrom
renovate/solhint-6.x
Open

chore(deps): update dependency solhint to v6#1469
renovate[bot] wants to merge 1 commit intomainfrom
renovate/solhint-6.x

Conversation

@renovate
Copy link
Contributor

@renovate renovate bot commented Jul 5, 2025

This PR contains the following updates:

Package Change Age Confidence
solhint (source) 5.0.36.0.3 age confidence

Release Notes

protofire/solhint (solhint)

v6.0.3

Compare Source

🛠️ Fix: removed unused files, normalized schema for validation, load-rules, base-checker and validator improvements

🛠️ Fix: removing console log from use-natspec rule (thanks brossetti1!!)

🛠️ Fix: misc minor issues and typos. General polish and stability

✨🛡️ Kudos to our contributors! 🛡️✨

v6.0.2

Compare Source

🛠️ Fix: corrected use-natspec enforcement behavior on internal functions

🛠️ Fix: misc minor issues and typos. General polish and stability

🆕 Rule: added foundry-no-block to detect usage of block.timestamp and block.number in Solidity tests

🆕 Rule: added no-unused-private-functions to prevent unused private function declarations

🆕 Rule: added no-immutable-before-declaration to enforce correct immutable declaration order

🧱 Enhancement: improvements to foundry-test-function rule, including naming and configuration updates

🧰 Infra: update GitHub Actions in CI workflows to v5 and v6. Keeps CI aligned with latest ecosystem changes

🧰 Infra: bump js-yaml dependency following security recommendations

🧹 Chore: stop publishing the test directory to npm packages. Reduces package size and noise

🧹 Chore: foundry-test-functions is deprecated and will be removed in v7.0.0. Please rename to foundry-test-function-naming.
WILL BE REPLACED IN v7

✨🛡️ Kudos to our contributors! 🛡️✨

v6.0.1

Compare Source

🛠️ Fix: no-unused-vars for modifiers

🛠️ Fix: no-unused-imports handled for multiline @​inheritdoc. Eliminates missed detections with multiline inheritance docs

🛠️ Fix: func-name-mixedcase improvement, added snake case exception in interface (naming). Aligns interface naming edge cases

🛠️ Fix: misc minor issues; docs link and typos. General polish and stability

🧰 Infra: Update GitHub Actions in CI workflows. Keeps CI up to date and consistent across jobs

🧰 Infra: Ignore enhancement to mimic as regular linters

🧹 Chore: Remove fs-extra dependency. Simplifies deps

🧹 Chore: Remove strip-ansi. Cleans unused/legacy deps

🧹 Chore: Changed changelog format

🧱 Enhancement: added config for no-unused-vars to validate parameters or not

🧱 Validation: Improve AJV schema validation and related caching approach. More robust/faster config validation

✨🛡️ Kudos to our contributors! 🛡️✨

v6.0.0

Compare Source

BREAKING CHANGE

Some rules were removed from the recommended ruleset and other were promoted. See below:
Rules revision - #​692

🟥 DEMOTED
  • payable-fallback: Outdated due to the introduction of receive(); rarely needed in modern Solidity.

✅ PROMOTED
  • interface-starts-with-i: Promotes clarity by clearly distinguishing interfaces from contracts.
  • gas-calldata-parameters: Encourages optimal memory usage for external functions.
  • gas-increment-by-one: Suggests using ++i style for gas-efficient increments.
  • gas-indexed-events: Improves event indexing for off-chain systems and reduces gas usage.
  • gas-small-strings: Recommends cheaper encoding using bytes32 for short strings.
  • gas-strict-inequalities: Helps avoid subtle off-by-one bugs in range conditions.
  • gas-struct-packing: Maximizes storage efficiency by suggesting tight struct packing.
  • duplicated-imports: Prevents redundant imports that bloat the AST and confuse code structure.
  • import-path-check: Ensures all imported files exist and resolve correctly, avoiding runtime errors.
  • function-max-lines: Encourages smaller, more readable and maintainable functions.
  • constructor-syntax: Enforces modern constructor syntax for consistency and clarity.
  • use-natspec: Promotes proper documentation with NatSpec for better audits and readability.
Fixed
Added
  • use-natspec: Promote the use of natspec and make several checks to enforce it - #​689



v5.2.0

Compare Source

Fixed
  • imports-order interference with no-unused-imports - #​684
Added
  • Added cache flag allowing to skip already evaluated files if no error was thrown and file did not change - #​685
  • Added multi-directory config support by hierarchy - #​678
  • Added better error handling on invalid configs - #​683

Thanks to @​smol-ninja - @​PaulRBerg @​lechpzn, @​otc-png, @​MamunC0der, @​kks-code, @​RidaMichofi

For making the time of proposing rules, pointing out typos, broken links, unused code, copyright, small reviews, etc. We really appreciate your contributions!



v5.1.0

Compare Source

Added
Updated
BREAKING CHANGE
  • Removed code for solhint:default ruleset - #​654

Thanks to @​fanqiaojun, @​maximevtush, @​MaxweLL22-22, @​PixelPil0t1, @​wgyt, @​operagxsasha, @​Hopium21, @​leopardracer, @​omahs, @​zeevick10, @​gap-editor, @​Olexandr88, @​jirutka

For making the time of pointing out typos, license update, broken links, unused code, copyright, small reviews, etc. We really appreciate your contributions!



v5.0.5

Compare Source

Fixed
  • gas-custom-errors #​620 - Support for Custom Errors inside require statements
  • compiler-version #​621 - Upgraded minimum requirement for the rule
  • reentrancy #​622 - Fixed path and typos
  • Typos #​623 - Fixed typos
  • Typo #​625 - Fixed typo
Added
  • New Rule: Duplicated Imports #​626
  • Cute Message on console report to gather community into discord channel



v5.0.4

Compare Source

Fixed
  • imports-order #​595 - Replaced single quotes with double quotes
  • gas-custom-errors #​613 - Allows the use of Requirement with Custom Errors
  • Typos and broken links #​611 - #​617
  • Upgraded solidity-parser dependency #​612
Added
  • .pre-commit-hooks.yaml to allow projects to run Solhint via pre-commit #​596 (Thanks to @​dbast)
  • Removed husky since it is not needed #​612




Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@vercel
Copy link

vercel bot commented Jul 5, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
dapp-starter Error Error Feb 12, 2026 2:53pm

@changeset-bot
Copy link

changeset-bot bot commented Jul 5, 2025

⚠️ No Changeset found

Latest commit: 9415100

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@gitstream-cm
Copy link

gitstream-cm bot commented Jul 5, 2025

🚨 gitStream Monthly Automation Limit Reached 🚨

Your organization has exceeded the number of pull requests allowed for automation with gitStream.
Monthly PRs automated: 251/250

To continue automating your PR workflows and unlock additional features, please contact LinearB.

@coderabbitai
Copy link

coderabbitai bot commented Jul 5, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@ellipsis-dev ellipsis-dev bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Important

Looks good to me! 👍

Reviewed everything up to 8d2c7e0 in 35 seconds. Click for details.
  • Reviewed 13 lines of code in 1 files
  • Skipped 1 files when reviewing.
  • Skipped posting 2 draft comments. View those below.
  • Modify your settings and rules to customize what types of comments Ellipsis leaves. And don't forget to react with 👍 or 👎 to teach Ellipsis.
1. package.json:59
  • Draft comment:
    Updated solhint to v6.0.0. Ensure that your solhint configuration is updated to reflect any breaking changes (e.g. removed or promoted rules) introduced in this version.
  • Reason this comment was not posted:
    Comment did not seem useful. Confidence is useful = 0% <= threshold 50% This comment is related to a dependency update, specifically the solhint version. It suggests ensuring that the configuration is updated to reflect any breaking changes. According to the rules, comments on dependency changes or library versions should be ignored, especially if they are not recognized. This comment seems to fall under that category.
2. package.json:59
  • Draft comment:
    Review solhint config files for compatibility with v6.0.0 changes.
  • Reason this comment was not posted:
    Comment did not seem useful. Confidence is useful = 0% <= threshold 50% This comment is asking the PR author to review configuration files for compatibility with a new version. It doesn't provide a specific suggestion or point out a specific issue in the code. It falls under the category of asking the author to ensure compatibility, which is against the rules.

Workflow ID: wflow_d52y7xsfiqhhLYBQ

You can customize Ellipsis by changing your verbosity settings, reacting with 👍 or 👎, replying to comments, or adding code review rules.

@vercel
Copy link

vercel bot commented Nov 11, 2025

Deployment failed with the following error:

Resource is limited - try again in 27 minutes (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/itman?upgradeToPro=build-rate-limit

@renovate renovate bot force-pushed the renovate/solhint-6.x branch from 246e7c1 to f017775 Compare December 3, 2025 19:51
@vercel
Copy link

vercel bot commented Dec 3, 2025

Deployment failed with the following error:

Resource is limited - try again in 2 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/itman?upgradeToPro=build-rate-limit

@socket-security
Copy link

socket-security bot commented Dec 3, 2025

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

@vercel
Copy link

vercel bot commented Dec 31, 2025

Deployment failed with the following error:

Resource is limited - try again in 2 minutes (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/itman?upgradeToPro=build-rate-limit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants