Skip to content

Lint smart contracts according to conventions - #7

Open
3esmit wants to merge 1 commit into
masterfrom
lint
Open

Lint smart contracts according to conventions#7
3esmit wants to merge 1 commit into
masterfrom
lint

Conversation

@3esmit

@3esmit 3esmit commented Dec 20, 2019

Copy link
Copy Markdown

Fixes errors in linting according to Solhint or Solium

  • indentation: Only use indent of 4 spaces.
  • quotes: "./KeycardWalletFactory.sol": Import statements must use double quotes only.

Fixes problem with different compilers by allowing a range of versions.

@3esmit
3esmit requested a review from gravityblast December 20, 2019 12:47
@3esmit

3esmit commented Dec 20, 2019

Copy link
Copy Markdown
Author

I recommend using https://github.com/juanfranblanco/vscode-solidity into https://github.com/VSCodium/vscodium/releases which would automatically warn about lint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant