On 20. 12. 2011 Bart Cerneels wrote:
> Just make sure you squash the commits that belong together :)

Do you mean some specific commits? E.g. there are 3 UpdateCapability removal-
related commits, but each one stands on its own (with commit message 
describing the specific change) and everything is perfectly bisectable. (I must 
say I'm not a big fan of squashing related changes - I instead prefer the 
Linux-kernel way of thinking with git: make very granular commits so that it 
is `git bisect` who finds erroneous changes, not you)

> And don't forget to Changelog in the same commit!

I guess you're alluding to my past commits. I've improved, I swear! :)

So it seems nobody opposes merging the patch-set, so I'll do it once I find 
another bit of time in this holiday time..

                Matěj
_______________________________________________
Amarok-devel mailing list
Amarok-devel@kde.org
https://mail.kde.org/mailman/listinfo/amarok-devel

Reply via email to