geode-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mbr...@apache.org
Subject incubator-geode git commit: GEODE-487: Remove integrationTest and distributedTest from build task.
Date Tue, 27 Oct 2015 17:56:42 GMT
Repository: incubator-geode
Updated Branches:
  refs/heads/develop 28eb3090b -> b2dea99cb


GEODE-487: Remove integrationTest and distributedTest from build task.

This commit removes integrationTest and distributedTest tasks from the
build task lifecycle. The tasks were associated with the 'check' task,
which is in the 'build' lifecycle. After removing the tasks, only the
'test' task is run when build task is executed.

Tested by running 'clean build'

Review available at https://reviews.apache.org/r/39664


Project: http://git-wip-us.apache.org/repos/asf/incubator-geode/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-geode/commit/b2dea99c
Tree: http://git-wip-us.apache.org/repos/asf/incubator-geode/tree/b2dea99c
Diff: http://git-wip-us.apache.org/repos/asf/incubator-geode/diff/b2dea99c

Branch: refs/heads/develop
Commit: b2dea99cbb909edf0ccca9d117147316c44e87bb
Parents: 28eb309
Author: Mark Bretl <mbretl@pivotal.io>
Authored: Mon Oct 26 11:26:54 2015 -0700
Committer: Mark Bretl <mbretl@pivotal.io>
Committed: Tue Oct 27 10:54:18 2015 -0700

----------------------------------------------------------------------
 build.gradle | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-geode/blob/b2dea99c/build.gradle
----------------------------------------------------------------------
diff --git a/build.gradle b/build.gradle
index 980eba8..7e477f3 100755
--- a/build.gradle
+++ b/build.gradle
@@ -409,10 +409,13 @@ subprojects {
     }
   })
 
-  check.dependsOn integrationTest
-  check.dependsOn distributedTest 
+  // Make precheckin task run all validation tests for checking in code.
+  task precheckin (dependsOn: [ build, integrationTest, distributedTest ]) {
+    description 'Run this task before checking in code to validate changes. This task combines
the following tasks: build, integrationTest, and distributedTest'
+  }
+
   check.dependsOn checkMissedTests
-  combineReports.mustRunAfter check, test, integrationTest, distributedTest,checkMissedTests
+  combineReports.mustRunAfter check, test, integrationTest, distributedTest, checkMissedTests
   build.finalizedBy combineReports
   check.finalizedBy combineReports
   test.finalizedBy combineReports


Mime
View raw message