Use ES6. If you see code in solium using es5, its probably old code (when I started writing the linter). Don't follow those bad habits.
Use const instead of let wherever possible.
Avoid the pyramid style of closing braces at all costs. Its a maintenance nightmare.
Be sure to run npm run lint once you're done writing your code. Fix any issues that arise.
If contributing rules, Please write a brief description of it in the PR conversation thread, so maintainers can instantly understand 1. The problem in solidity that this rule is solving 2. Brief on how it is solving it.
Before making any significant changes to the codebase in the core linter (eg- adding new utility methods, new core feature, etc), PLEASE discuss over an issue first. This saves your efforts from going waste.
Things to put in guidelines:
ES6
. If you see code in solium using es5, its probably old code (when I started writing the linter). Don't follow those bad habits.const
instead oflet
wherever possible.npm run lint
once you're done writing your code. Fix any issues that arise.develop
branch.