-
Notifications
You must be signed in to change notification settings - Fork 397
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Integrate helloworld project with Travis CI #212
Conversation
Codecov Report
@@ Coverage Diff @@
## master #212 +/- ##
==========================================
+ Coverage 80.63% 80.64% +<.01%
==========================================
Files 323 323
Lines 10350 10350
Branches 536 536
==========================================
+ Hits 8346 8347 +1
+ Misses 2004 2003 -1
Continue to review full report at Codecov.
|
@@ -53,6 +55,8 @@ configurations { | |||
dependencies { | |||
// Scala | |||
zinc 'com.typesafe.zinc:zinc:0.3.15' | |||
scoverage "org.scoverage:scalac-scoverage-plugin_$scalaVersion:$scoveragePluginVersion" |
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.
you also need
testCompile "org.scoverage:scalac-scoverage-plugin_$scalaVersion:$scoveragePluginVersion"
testCompile "org.scoverage:scalac-scoverage-runtime_$scalaVersion:$scoveragePluginVersion"
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.
fixed
@@ -68,3 +72,10 @@ jobs: | |||
echo -e "Test files running on this container : `eval $COUNT_TEST_FILE_CMD`\n$TEST_FILES\n" | |||
./gradlew reportScoverage | |||
name: 'Build Part 4' | |||
- script: | |
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.
simply do:
- script: |
cd helloworld && ./gradlew reportScoverage
name: 'Build - Helloworld'
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.
@tovbinm fixed
Related issues
#114
Describe the proposed solution
Added helloworld samples to the travis build
Describe alternatives you've considered
none
Additional context
none