solhint
solhint copied to clipboard
Update examples in the "Configure linter with comments" section
At the very least, the rule used for those examples is deprecated, but the whole section could probably use a revamp too.
Section mentioned is this one - https://github.com/protofire/solhint/blob/master/docs/configuration.md#configure-linter-with-comments
Approval on issue is from @fvictorio help on writing side from @chrischinchilla
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
This issue now has a funding of 100.0 DAI (100.0 USD @ $1.0/DAI) attached to it as part of the ConsenSys fund.
- If you would like to work on this issue you can 'start work' on the Gitcoin Issue Details page.
- Want to chip in? Add your own contribution here.
- Questions? Checkout Gitcoin Help or the Gitcoin Slack
- $132,648.27 more funded OSS Work available on the Gitcoin Issue Explorer
@fvictorio can you please explain what do you mean by "the rule used for those examples is deprecated", because i am not sure to understand well if we are talking about the same thing.
throw
is indeed deprecated in solidity but this is why this rule exists, a warning to the user who tries to use this old solidity feature.
So the example in your doc is fine.
/* solhint-disable avoid-throw */
if (a > 1) {
throw;
}
/* solhint-enable avoid-throw */
By reading the code the comment directive is fine and works well, it avoids to trigger the "throw" is deprecated, avoid to use it.
warning. code here
Maybe you would mean that we should use other kind of examples (maybe not critical security rules) with or for replacing this one ?
Oops, sorry, my bad. I was talking specifically about the compiler-fixed
and compiler-gt-0_4
rules, the other ones seem fine.
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
Work for 100.0 DAI (100.0 USD @ $1.0/DAI) has been submitted by:
@datakarima please take a look at the submitted work:
- PR by @akersof
- Learn more on the Gitcoin Issue Details page
- Want to chip in? Add your own contribution here.
- Questions? Checkout Gitcoin Help or the Gitcoin Slack
- $132,378.80 more funded OSS Work available on the Gitcoin Issue Explorer
Issue Status: 1. Open 2. Started 3. Submitted 4. Done
The funding of 100.0 DAI (100.0 USD @ $1.0/DAI) attached to this issue has been approved & issued to @akersof.
- Learn more on the Gitcoin Issue Details page
- Questions? Checkout Gitcoin Help or the Gitcoin Slack
- $130,170.83 more funded OSS Work available on the Gitcoin Issue Explorer