You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mxnet.apache.org by ma...@apache.org on 2018/09/20 12:48:06 UTC

[incubator-mxnet] 01/01: Correct PR branch detection in code coverage

This is an automated email from the ASF dual-hosted git repository.

marcoabreu pushed a commit to branch fix-testcoverage-report
in repository https://gitbox.apache.org/repos/asf/incubator-mxnet.git

commit 2c8956aa5aea401c6d49ccf2f704da1c35dba0af
Author: Marco de Abreu <ma...@gmail.com>
AuthorDate: Thu Sep 20 14:46:55 2018 +0200

    Correct PR branch detection in code coverage
---
 ci/Jenkinsfile_utils.groovy | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ci/Jenkinsfile_utils.groovy b/ci/Jenkinsfile_utils.groovy
index ca5f234..b50a6da 100644
--- a/ci/Jenkinsfile_utils.groovy
+++ b/ci/Jenkinsfile_utils.groovy
@@ -78,7 +78,7 @@ echo ${libs} | sed -e 's/,/ /g' | xargs md5sum
 def publish_test_coverage() {
     // CodeCovs auto detection has trouble with our CIs PR validation due the merging strategy
     def codecovArgs = ""
-    if (env.CHANGE_ID != '') {
+    if (env.CHANGE_ID) {
       // PR execution
       // Take the previous commit because of our PR merge strategy that adds a temporary commit for CI
       GIT_COMMIT_HASH = sh (script: "git rev-parse @~", returnStdout: true)