You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2020/02/07 06:44:42 UTC
[GitHub] [incubator-shardingsphere] kimmking opened a new pull request
#4197: Travis config
kimmking opened a new pull request #4197: Travis config
URL: https://github.com/apache/incubator-shardingsphere/pull/4197
Polish travis config.
Changes proposed in this pull request:
- modify MaxPermSize to MaxMetaspaceSize
- add maven opt "--batch-mode --no-transfer-progress" to make build faster and optimize log info
- add after_success step to report coverage to codecov.io
- add after_failure step to print failure info
- merge travis timeout config to build script step
- remove unused parameter "sudo" according to travis console prompt info
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] kimmking commented on a change in pull
request #4197: Travis config
Posted by GitBox <gi...@apache.org>.
kimmking commented on a change in pull request #4197: Travis config
URL: https://github.com/apache/incubator-shardingsphere/pull/4197#discussion_r376240759
##########
File path: .travis.yml
##########
@@ -1,15 +1,21 @@
-sudo: required
-cache:
- directories:
- - "$HOME/.m2"
-
language: java
jdk:
- openjdk8
+install: true
+
+cache:
+ directories:
+ - "$HOME/.m2"
+
before_script:
- - echo "MAVEN_OPTS='-Xmx1024m -XX:MaxPermSize=256m'" > ~/.mavenrc
+ - echo "MAVEN_OPTS='-Xmx1024m -XX:MaxMetaspaceSize=256m'" > ~/.mavenrc
+
script:
- - mvn clean install cobertura:cobertura coveralls:report -DrepoToken="${COVERALLS_REPO_TOKEN}" -Dmaven.javadoc.skip=true
+ - travis_wait 30 mvn --batch-mode --no-transfer-progress clean install cobertura:cobertura coveralls:report -DrepoToken="${COVERALLS_REPO_TOKEN}" -Dmaven.javadoc.skip=true
+
+after_success:
+ - bash <(curl -s https://codecov.io/bash)
-install: travis_wait 30 mvn install --quiet
+after_failure:
+ - echo "build failed!"
Review comment:
ok
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] tuohai666 commented on issue #4197:
Travis config
Posted by GitBox <gi...@apache.org>.
tuohai666 commented on issue #4197: Travis config
URL: https://github.com/apache/incubator-shardingsphere/pull/4197#issuecomment-583259296
Please add labels and milestone.
I think this pr is also applicable for 4.0.1 branch.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] tuohai666 merged pull request #4197:
Travis config
Posted by GitBox <gi...@apache.org>.
tuohai666 merged pull request #4197: Travis config
URL: https://github.com/apache/incubator-shardingsphere/pull/4197
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [incubator-shardingsphere] tuohai666 commented on a change in pull
request #4197: Travis config
Posted by GitBox <gi...@apache.org>.
tuohai666 commented on a change in pull request #4197: Travis config
URL: https://github.com/apache/incubator-shardingsphere/pull/4197#discussion_r376237092
##########
File path: .travis.yml
##########
@@ -1,15 +1,21 @@
-sudo: required
-cache:
- directories:
- - "$HOME/.m2"
-
language: java
jdk:
- openjdk8
+install: true
+
+cache:
+ directories:
+ - "$HOME/.m2"
+
before_script:
- - echo "MAVEN_OPTS='-Xmx1024m -XX:MaxPermSize=256m'" > ~/.mavenrc
+ - echo "MAVEN_OPTS='-Xmx1024m -XX:MaxMetaspaceSize=256m'" > ~/.mavenrc
+
script:
- - mvn clean install cobertura:cobertura coveralls:report -DrepoToken="${COVERALLS_REPO_TOKEN}" -Dmaven.javadoc.skip=true
+ - travis_wait 30 mvn --batch-mode --no-transfer-progress clean install cobertura:cobertura coveralls:report -DrepoToken="${COVERALLS_REPO_TOKEN}" -Dmaven.javadoc.skip=true
+
+after_success:
+ - bash <(curl -s https://codecov.io/bash)
-install: travis_wait 30 mvn install --quiet
+after_failure:
+ - echo "build failed!"
Review comment:
Add an empty line at the end of file.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services