Re: Dropping '+' from fetch = +refs/heads/*:refs/remotes/origin/*?
- From
Jeff King <peff@peff.net>
- Date
- Sep 5, 2011, 20:57 UTC
- Message-ID
- <20110905205735.GA5578@sigill.intra.peff.net>
- In-Reply-To
- <CAJo=hJtvU+ujYBMvrgVJpBdaTUq+NOsQwVFkL-A4pHv-CRPdDg@mail.gmail.com>
On Mon, Sep 05, 2011 at 01:53:42PM -0700, Shawn O. Pearce wrote:
Show 9 quoted lines
> > Sure. I'm totally open to the idea of making the non-fast-forward > > warning more obvious. Suggestions for wording (though I am tempted by > > "HEY STUPID" above ;) )? > > I'm not suggesting all non-fast-forward should issue a bigger warning. > pu updates daily with a non-fast-forward. That isn't useful. > > But if the local reflog hints that this reference almost never does a > non-fast-forward, and then it does, that should be a big warning.
Right. What I mean is, what should the bigger warning look like?
Also, you suggested caching to avoid looking through the whole reflog each time. I think you could probably just sample the last 10 or so reflog entries to get an idea.
-Peff