question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

bug: solhint ordering

See original GitHub issue

"solhint": "3.3.2",

"ordering": "error",

doesnt seem to work e.g here.

Issue Analytics

  • State:closed
  • Created 3 years ago
  • Comments:7 (3 by maintainers)

github_iconTop GitHub Comments

2reactions
gitpushacommented, Dec 17, 2020

Hi @fernandomg thanks for your time!

doesnt seem to work e.g here. Also the initial issue what seems to be an actual bug in the existing rule set because solhint doesnt complain about public view coming before external payable here for example.

You can clone the repo and run yarn lint:sol to see that it doesnt complain. I noticed this deficiency in the ordering rule in many other places too.

Also, you guys should open a gitcoin grant 😃

1reaction
gitpushacommented, Mar 21, 2021

thx for the merge @fernandomg 👍

Read more comments on GitHub >

github_iconTop Results From Across the Web

ordering Solhint - GitHub Pages
This rule accepts a string option of rule severity. Must be one of “error”, “warn”, “off”. Default to warn. Example Config. { ...
Read more >
solhint/ordering.md at master - rules - GitHub
This rule accepts a string option of rule severity. Must be one of "error", "warn", "off". Default to warn. Example Config.
Read more >
Developers - bug: solhint ordering - - Bountysource
"ordering": "error",. doesnt seem to work e.g here. See More. View in GitHub. SOLVE ISSUE.
Read more >
protofire/solhint | Coveralls - Test Coverage History & Statistics
Test code coverage history for protofire/solhint. ... LAST BUILD BRANCH: fix/274-ordering ... Better error when config file doesnt exist Embed ▾ ...
Read more >
CoorestOfficialToken.sol - Smart Contract Audit
potential issues like race conditions, transaction-ordering dependence, ... Platforms and Tools: Remix IDE, Solhint, VScode, Slither, Mythril.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found