You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Hi @maraoz ,
Its been quite a while since Soliumv1 was released. It comes with significant improvements including automatic code fixing, new security checks for contract code and ability to configure rule behavior (like 2 spaces instead of 4).
I'm raising this issue to simply make you aware, since v1 will significantly improve your workflow.
Adoption is pretty simple:
npm i -g solium
Going by your current .soliumrc.json, your new .soliumrc.json would look like:
Please do not hesitate in closing this issue if you feel its not currently a priority. However, I do recommend that you upgrade to further improve contract security. Feel free to ask me anything!
(I can make a PR for this, but a dev must update their solium version on workstation too)
The text was updated successfully, but these errors were encountered:
Hi @maraoz ,
Its been quite a while since Solium
v1
was released. It comes with significant improvements including automatic code fixing, new security checks for contract code and ability to configure rule behavior (like 2 spaces instead of 4).I'm raising this issue to simply make you aware, since v1 will significantly improve your workflow.
Adoption is pretty simple:
npm i -g solium
Going by your current
.soliumrc.json
, your new.soliumrc.json
would look like:(Although I should inform you that the mixedcase rule's bugs have been resolved)
If, for eg, you'd like to disable a security rule you don't agree with, simply add
"security/rule-name": "off"
insiderules
object.Run normally using
solium -d contracts/
orsolium -d contracts/ --fix
to apply code fixes (wherever possible).Usage docs
List of Security Rules
Please do not hesitate in closing this issue if you feel its not currently a priority. However, I do recommend that you upgrade to further improve contract security. Feel free to ask me anything!
(I can make a PR for this, but a dev must update their solium version on workstation too)
The text was updated successfully, but these errors were encountered: