Re: [infinispan-dev] Reminders about GIT
On 27 October 2011 14:22, Manik Surtani wrote: > Sanne - you should add this as a warning section > to https://docs.jboss.org/author/display/ISPN/Contributing+-+Source+Control > :-) Done ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Reminders about GIT
+1 ROFL! --Kevin On 27 October 2011 14:22, Manik Surtani wrote: > Sanne - you should add this as a warning section to > https://docs.jboss.org/author/display/ISPN/Contributing+-+Source+Control > :-) > > On 18 Oct 2011, at 16:09, Sanne Grinovero wrote: > > 1) If you have warnings about "Merge made by recursive" you have to > fix it rebasing. > > 2) If you have warnings about "non-fast-forward" you have to rebase. > > 3) If you see "non-fast-forward updates were rejected" you shall never > use "force" on upstream! It means that another patch was merged before > you and you have to update your master again, and rebase again. > > 4) "force" is allowed only in special maintenance circumstances. If > you find you're needing it to handle a pull request, then you're doing > it wrong, and the mistake might be a dangerous one. It's like the good > rule of never commit when you're drunk (coding is allowed). > ___ > infinispan-dev mailing list > infinispan-dev@lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev > > > -- > Manik Surtani > ma...@jboss.org > twitter.com/maniksurtani > > Lead, Infinispan > http://www.infinispan.org > > > > > ___ > infinispan-dev mailing list > infinispan-dev@lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev > ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Reminders about GIT
Sanne - you should add this as a warning section to https://docs.jboss.org/author/display/ISPN/Contributing+-+Source+Control :-) On 18 Oct 2011, at 16:09, Sanne Grinovero wrote: > 1) If you have warnings about "Merge made by recursive" you have to > fix it rebasing. > > 2) If you have warnings about "non-fast-forward" you have to rebase. > > 3) If you see "non-fast-forward updates were rejected" you shall never > use "force" on upstream! It means that another patch was merged before > you and you have to update your master again, and rebase again. > > 4) "force" is allowed only in special maintenance circumstances. If > you find you're needing it to handle a pull request, then you're doing > it wrong, and the mistake might be a dangerous one. It's like the good > rule of never commit when you're drunk (coding is allowed). > ___ > infinispan-dev mailing list > infinispan-dev@lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Reminders about GIT
On 18 Oct 2011, at 17:50, Mircea Markus wrote: >> It's like the good rule of never commit when you're drunk (coding is >> allowed). > ROFL! Excellent! :-) -- Manik Surtani ma...@jboss.org twitter.com/maniksurtani Lead, Infinispan http://www.infinispan.org ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Reminders about GIT
> It's like the good rule of never commit when you're drunk (coding is > allowed). ROFL! ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Reminders about GIT
Nice! On Tue, Oct 18, 2011 at 6:16 PM, Sanne Grinovero wrote: > Almost forgot the main tip; use: > > git config branch.master.mergeoptions "--ff-only" > > It will prevent a non-linear creating merge with an error as > > "fatal. Not possible to fast-forward, aborting." > > Cheers, > Sanne > > On 18 October 2011 16:09, Sanne Grinovero wrote: >> 1) If you have warnings about "Merge made by recursive" you have to >> fix it rebasing. >> >> 2) If you have warnings about "non-fast-forward" you have to rebase. >> >> 3) If you see "non-fast-forward updates were rejected" you shall never >> use "force" on upstream! It means that another patch was merged before >> you and you have to update your master again, and rebase again. >> >> 4) "force" is allowed only in special maintenance circumstances. If >> you find you're needing it to handle a pull request, then you're doing >> it wrong, and the mistake might be a dangerous one. It's like the good >> rule of never commit when you're drunk (coding is allowed). >> > ___ > infinispan-dev mailing list > infinispan-dev@lists.jboss.org > https://lists.jboss.org/mailman/listinfo/infinispan-dev > ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
Re: [infinispan-dev] Reminders about GIT
Almost forgot the main tip; use: git config branch.master.mergeoptions "--ff-only" It will prevent a non-linear creating merge with an error as "fatal. Not possible to fast-forward, aborting." Cheers, Sanne On 18 October 2011 16:09, Sanne Grinovero wrote: > 1) If you have warnings about "Merge made by recursive" you have to > fix it rebasing. > > 2) If you have warnings about "non-fast-forward" you have to rebase. > > 3) If you see "non-fast-forward updates were rejected" you shall never > use "force" on upstream! It means that another patch was merged before > you and you have to update your master again, and rebase again. > > 4) "force" is allowed only in special maintenance circumstances. If > you find you're needing it to handle a pull request, then you're doing > it wrong, and the mistake might be a dangerous one. It's like the good > rule of never commit when you're drunk (coding is allowed). > ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev
[infinispan-dev] Reminders about GIT
1) If you have warnings about "Merge made by recursive" you have to fix it rebasing. 2) If you have warnings about "non-fast-forward" you have to rebase. 3) If you see "non-fast-forward updates were rejected" you shall never use "force" on upstream! It means that another patch was merged before you and you have to update your master again, and rebase again. 4) "force" is allowed only in special maintenance circumstances. If you find you're needing it to handle a pull request, then you're doing it wrong, and the mistake might be a dangerous one. It's like the good rule of never commit when you're drunk (coding is allowed). ___ infinispan-dev mailing list infinispan-dev@lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev