-
Notifications
You must be signed in to change notification settings - Fork 962
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add Solidity Linter to Travis #376
Conversation
.travis.yml
Outdated
- os: linux | ||
env: V=0.15.0 | ||
before_install: | ||
# Install Bazel | ||
# Install Bazel and initialize solidity linter at the end |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert this comment. The comment reflects what happens in this next step
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add your npm step after the bazel installation step.
On line 51 add:
# Install solidity linter
- npm install -g solium
.travis.yml
Outdated
@@ -9,11 +9,11 @@ matrix: | |||
- lint | |||
script: | |||
- | |||
go get github.com/alecthomas/gometalinter && gometalinter --install && gometalinter ./... --deadline=10m --exclude=client/internal/client_helper.go | |||
npm install -g solium && go get github.com/alecthomas/gometalinter && gometalinter --install && gometalinter ./... --deadline=10m --exclude=client/internal/client_helper.go |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Revert this. You're installing solium in the linter container and it won't be available to to the testing container.
.travis.yml
Outdated
@@ -71,6 +71,9 @@ matrix: | |||
# Check that target visibility is correct.. | |||
- ./check_visibility.sh | |||
|
|||
# Check solidity linter. | |||
- solium -d contracts/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me
.travis.yml
Outdated
@@ -71,6 +71,9 @@ matrix: | |||
# Check that target visibility is correct.. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you fix this double .
? Thanks!
Another thing you could do is to add another entry to the matrix. - os: linux
language: node_js
env:
- solidity
node_js:
- "lts/*"
before_install:
- npm install -g solium
install: true # Skip npm install
script:
# Check solidity linter.
- solium -d contracts/ This build would run in parallel rather than sequential in the test container. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you have some yaml issues. Travis can鈥檛 parse this config, it seems.
First time playing Travis 馃槄 Apologize in advance for any obvious mistake. Let me know if anything is worng