openstack team mailing list archive

[Christian Berendt <berendt@xxxxxxxxxxxxx>]

Hi.

Sometimes there are very useless commit messages after merging a
propsal into trunk.

Example:

" 748. Did a pull from trunk to be sure I had the latest, then deleted
the test directory. I guess it appeared when I started using venv. Doh.
(detail)"

or my own:

" 747. reverted my changes from
https://code.launchpad.net/~berendt/nova/lp722554/+merge/50579 and
reused the existing db api methods to add the disabled services. Looks
much better now :)

i couldn't figure out how to uncommit and revert my changes in an
already pushed branch, always ending in "ERROR: These branches
have diverged.". So i created a new branch rev2 and a new
proposal... (detail)"

I think the problem is that per default the last commit message in the
branch is used.

We should always set a correct commit message in the merge propsal and
we should don't merge code with useless commit messages.

Bye, Christian.

***
Visit us at CeBIT 2011!
At our stand located in the Open Source Park in hall 2, booth-no.:
D44/133 we demonstrate "Virtual System Cluster" and "Infrastructure
Cloud with OpenStack" setups.
On Wedensday between 9:00am and 11:59am we invite you to a traditional
Bavarian breakfast at our stand.
***

-- 
Christian Berendt
Linux / Unix Consultant & Developer
Tel.: +49-171-5542175
Mail: berendt@xxxxxxxxxxxxx

B1 Systems GmbH
Osterfeldstraße 7 / 85088 Vohburg / http://www.b1-systems.de
GF: Ralph Dehner / Unternehmenssitz: Vohburg / AG: Ingolstadt,HRB 3537

Attachment: signature.asc
Description: PGP signature