Integrate with Solium linter #17
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
@denisglotov this one is for you
This PR adds integration of Emacs solidity mode and flycheck with the solium linter.
At this point you now need to specify the path to the
solc
or the path to thesolium
binary viasolidity-solc-path
orsolidity-solium-path
respectively. Reason is that in some configurations, solc may be added to the path after emacs is launched (e.g. systemd) so it's always good practise to have it explicitly specified.Moreover since now we can have multiple flycheck checkers you need to specify the active flycheck checker. That is done via the
solidity-flycheck-active-checker
option which can either be"solc"
or"solium"
.Finally check the README for possible configuration options of solium and solc.