+1 . Initial dev experience needs to be frictionless... :)
________________________________________ From: Bruce Schuchardt <bschucha...@pivotal.io> Sent: Thursday, October 1, 2015 4:56 PM To: dev@geode.incubator.apache.org Subject: Re: Propose changing check to only depend on UnitTest +1 Le 10/1/2015 2:21 PM, Darrel Schneider a écrit : > +1 > > How about "precheckin" for the target that runs IntegrationTest and > DistributionTest? Perhaps this target should start with a line informing > the runner that it will take hours. > > > On Thu, Oct 1, 2015 at 12:48 PM, Kirk Lund <kl...@pivotal.io> wrote: > >> Proposal: >> >> 1) Remove IntegrationTest and DistributionTest from "check" target >> >> 2) Add a new target (name TBD) that executes IntegrationTest and >> DistributionTest >> >> Reasons: >> >> 1) new people who clone and then build wait forever and complain that build >> hung resulting in bad first impression by users >> >> 2) Index-Geodein5minutes instructs user to skip tests but internet wisdom >> says never use -Dskip.tests=true >> >> >> https://cwiki.apache.org/confluence/display/GEODE/Index#Index-Geodein5minutes >> >> 3) non-developers don't need to run IntegrationTest and DistributedTest -- >> they just want to clone, build and try out basics to see if they want to >> become a Geode User >> >> 4) anyone who decides they want to become a developer can easily learn that >> we invoke a target to run the long-running tests -- we can easily document >> this everywhere if people are truly worried about it >> >> 5) this change encourages us to improve our coverage with fast-running >> UnitTests >> >> Anti-goals >> >> 1) I'm not suggesting that anyone can commit changes without running the >> long-running tests >> >> -Kirk >>
