Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update dependency solhint to v5 #125

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented May 12, 2024

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
solhint (source) ^4.1.1 -> ^5.0.3 age adoption passing confidence

Release Notes

protofire/solhint (solhint)

v5.0.3

Compare Source

Fixed



v5.0.2

Compare Source

Fixed
Added



v5.0.1

Compare Source

BREAKING CHANGES (refer to v5.0.0)

Fixed an issue on the returining values where only was evaluating the first report instead of all of them.



v5.0.0

Compare Source

BREAKING CHANGES
Solhint EXIT codes

Solhint changed how the exit codes are implemented:

Exit with 0 code When execution was ok and there were no errors when evaluating the code according to the rules

Exit with 1 code When execution was ok and there are errors reported

Exit with 1 code When execution was ok and max warnings is lower than the reported warnings

Exit with 255 code When there's an error in the execution (bad config, writing not allowed, wrong parameter, file not found, etc)

Solhint QUIET mode

QUIET mode (-c quiet) option now works with the warnings and may exit with 1 if there are more than defined by user

Thanks to @​juanpcapurro for providing the code



v4.5.4

Compare Source

Fixed
  • gas-custom-errors improved logic to ranged pragma versions #​573
  • gas-indexed-events #​573

v4.5.3

Compare Source

v4.5.2

Compare Source

Updated
  • Update Readme file to include all autofix rules
  • Update docker file
  • Update package json file

v4.5.1

Compare Source

v4.5.0

Compare Source

Updated
  • Update dependencies in package json #​548
  • Custom errors rules checks from 0.8.4 forward before warning #​555
  • Parser support up to Solidity 0.8.22 (*)

(*) Note: Solhint can work with latest Solidity versions.
If new grammar/syntax is added or changed, it could give some false positives or miss a rule.
But overall functionality will work as expected.

Added
  • New Rule: Interface starts with i #​557
Gas Consumption Rules
  • New Rule: [GC] Mutlitoken1155 rule #​541
  • New Rule: [GC] Small strings check #​542
  • New Rule: [GC] Indexed events #​543
  • New Rule: [GC] Calldata parameters #​544
  • New Rule: [GC] Increment by one #​545
  • New Rule: [GC] Struct packing #​546
  • New Rule: [GC] Name Return Values #​552
  • New Rule: [GC] Custom Errors #​553 #​555
  • New Rule: [GC] Dot Length in Loops #​559
  • New Rule: [GC] Gas Strict Inequalities #​560
Fixed
  • explicit-types logic improved and bug free #​551
  • payable fallback Improved behavior #​561
BREAKING CHANGES
  • named-return-values rule was renamed to gas-named-return-values and now it is part of Gas Consumption ruleset #​552
  • custom-errors rule was renamed to gas-custom-errors and now it is part of Gas Consumption ruleset #​553
  • Return error 0 when executed correctly #​554
  • Default severity modified to WARN instead of OFF for
    foundry-test-functions and named-parameters-mapping rules #​556

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.

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