solhint error/wraning not showing up
See original GitHub issueI wanted to switch to solhint for linting my code but vscode-solidity does not show any error or warning when I configure solhint via “solidity.linter” : “solhint” config.
When I run solhint via
solhint -f table src/**/*.sol
it shows a lot of warning and errors
Also when I use solium, it shows the error correctly
What can be wrong ?
Issue Analytics
- State:
- Created 4 years ago
- Comments:7 (2 by maintainers)
Top Results From Across the Web
solhint
Start using solhint in your project by running `npm i solhint`. There are 45 other projects in the npm registry using solhint.
Read more >Solhint - GitHub Pages
Solhint is an open source project created by https://protofire.io. Its goal is to provide a linting utility for Solidity code.
Read more >Solhint-Prettier Plugin - Static Analysis Tools
The Solhint Prettier plugin allows you to connect the Prettier formatting rules to the Solhint linter. Used as a Solidity linter, Solhint detects...
Read more >Intro to Solidity Linting and Formatting
A developer workflow for Solhint, Prettier-Solidity and Git Hooks. ... This tutorial goes over one of the many ways to set them up....
Read more >solhint-plugin-prettier - npm package
All security vulnerabilities belong to production dependencies of direct and indirect packages. License: MIT. Security Policy: No. We found ...
Read more >Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start FreeTop Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found
Top GitHub Comments
Hmmm - I’m on 0.0.72 now and the plugin is still not adhering to
solhint:recommended
items… For instance if you checkout our code (https://github.com/Synthetixio/synthetix),npm install
, then lookupcontracts/LimitedSetup.sol
in VSCode say, you won’t see warnings. Yet if you runnpx solhint contracts/*.sol
you’ll seestate-visibility
warnings. If you add"state-visibility": "warn"
to.solhint.json
then the warnings come up in vscode as expected.@justinjmoses thanks for that, ill put that in the next release. The expectation is to have just the rules: https://github.com/juanfranblanco/vscode-solidity/blob/master/src/linter/solhint.ts#L66-L71