[petsc-dev] testing before merging to master

Matthew Knepley knepley at gmail.com
Sun Nov 24 18:19:52 CST 2013


On Sun, Nov 24, 2013 at 6:02 PM, Barry Smith <bsmith at mcs.anl.gov> wrote:

>
> On Nov 24, 2013, at 2:47 PM, Jed Brown <jedbrown at mcs.anl.gov> wrote:
>
> > Barry Smith <bsmith at mcs.anl.gov> writes:
> >>  Jed and I may disagree on this but I believe that if your branch is
> >>
> >> 1) done (that is would be useful for users)
> >>
> >> 2) is completely clean in next
> >>
> >> 3) satisfies PETSc coding standards
> >>
> >> then it should be merged into master and not just “hang around” in
> >> next for days.
> >
> > My reservation is that if you are not confident about the test suite
> > being complete,
>
>     Hanging around in next is not going to make a test suite magically
> become better!!!!  So I would put that in the category of 1) the branch is
> not done because part of its test suite is missing (note that the test
> suite is suppose to be part of a complete branch).
>
> > or if a corresponding change for a dependent package
> > (like petsc4py) is in the works,
>
>    Again 1) the branch is not complete.
>
> >>
> >> then anyone could easily keep track of the two types of merges and
> >> they could be handled properly without requiring someone to remember
> >> something. Is there anyway to do this?
> >
> > How about if the person merging to 'next' pastes the TODO list for the
> > incomplete branch into the commit message.
>
>   Ok, let’s formalize this and you tell us how to automate finding the
> “TODO” list for these branches:
>
>    1) What is the format of the “TODO” list?  For example
> “Todo-before-merge-to-master: ….” ?
>
>    2) What git command do I use to list all branches merged into next (but
> not master) with todo lists and what is in each todo’s list?
>
>       For example, I run   gitmyaliasforfindingincompletebranchesinnext
>  and it produces
>
>            barry/cool-new-newthing : todo - test suite that handles 5
> dimensional case
>            satish/a-feature : todo - provide options database options for
> this feature
>            jed/c-feature : todo - use feature in SLEPc
>
>    Now we have no way of tracking all this stuff and that is bad.


I think we need a better automated way to notify people that checked in
something that a build failed since
there a bunch of builds to check. Can we get the branches merged since the
last build and on any failure/warning
we send the brnach owner an email?

   Matt


>
>   Barry
>

-- 
What most experimenters take for granted before they begin their
experiments is infinitely more interesting than any results to which their
experiments lead.
-- Norbert Wiener
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mcs.anl.gov/pipermail/petsc-dev/attachments/20131124/7e744407/attachment.html>


More information about the petsc-dev mailing list