Skip to content
This repository has been archived by the owner on Jul 11, 2019. It is now read-only.

tests: add and follow solium rules #154

Open
wants to merge 9 commits into
base: master
Choose a base branch
from

Conversation

come-maiz
Copy link
Contributor

Brought to you with 🤓 by the #static task force.
Remember that if you have $opinions, you should report an issue on https://github.com/ZeppelinSolutions/code-style where we will discuss about it and then update the rules accordingly, if required.

@come-maiz
Copy link
Contributor Author

@facuspagnuolo rules updated to 120 lines.

@facuspagnuolo facuspagnuolo added the status:blocked Blocked issue label Jun 7, 2018
Copy link
Contributor

@facuspagnuolo facuspagnuolo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Waiting for the new approach we discussed offline to tackle linter rules

.soliumrc.json Outdated
"quotes": ["error", "double"],
"no-empty-blocks": "off",
"indentation": ["error", 2],
"max-len": ["error", 79],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's use 120

@come-maiz come-maiz force-pushed the tests/solidity-lint branch from 2b6bd98 to 2daf329 Compare June 12, 2018 19:47
@come-maiz
Copy link
Contributor Author

@facuspagnuolo this is ready updating the style rules as warnings.
Please check the package-lock.json and let me know if I generated it correctly.

@coveralls
Copy link

Pull Request Test Coverage Report for Build 614

  • 5 of 5 (100.0%) changed or added relevant lines in 2 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 97.122%

Totals Coverage Status
Change from base Build 610: 0.04%
Covered Lines: 97
Relevant Lines: 97

💛 - Coveralls

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status:blocked Blocked issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants