[petsc-dev] strangness in Chebyshev estimate of eigenvalues
Satish Balay
balay at mcs.anl.gov
Tue May 24 13:20:40 CDT 2016
Well 'git merge master' on this branch doesn't help with this concern.
The process is
- don't merge to next until feature complete
[in this stage - you can rebase against master as many times as necessary]
- on feature completion - merge to next
- and monitor 'next' tests dashboard - until all related issues are
resolved and then merge to master [this next testing should be
resolved in a few days - not weeks]
- [you can run a local test to make sure this is working before you
push]
- and if issues come up in 'master' tests on dashboard - fix in
feature branch - and merge to master.
Satish
On Tue, 24 May 2016, Mark Adams wrote:
> OK, so you just trust that working in next is proof that it will work in
> master and just 'git merge branch' in master without ever doing a 'git
> merge master' in branch.
>
> Do I now wait for my pull request to be approved?
>
> On Tue, May 24, 2016 at 12:12 AM, Jed Brown <jed at jedbrown.org> wrote:
>
> > Mark Adams <mfadams at lbl.gov> writes:
> > > I am merging master more than I need to, just to be up to date. I guess
> > it
> > > pollutes the history with all these merges. And I should squash them...
> >
> > You shouldn't merge from 'master' unless you need something specific,
> > and in that case, write a commit message explaining what it is that you
> > need. You can rebase at will (before publishing your branch), but it's
> > generally fine to just work in your branch and ignore what is happening
> > in other branches.
> >
> > The rationale is explained in detail at the links in this section.
> >
> >
> > https://bitbucket.org/petsc/petsc/wiki/developer-instructions-git#markdown-header-merging
> >
>
More information about the petsc-dev
mailing list