[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [AUCTeX-devel] patch with improved commit message
From: |
Tassilo Horn |
Subject: |
Re: [AUCTeX-devel] patch with improved commit message |
Date: |
Sat, 17 Sep 2016 22:37:16 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux) |
Uwe Brauer <address@hidden> writes:
> > Oh, and "Correct incorrect filling" is not a good commit message
> > for a patch against auctex. That's part of the history in your
> > repository while creating the final patch. We want just the very
> > final, perfectly shaped end result. No one needs to know that
> > you messed up double-space sentence ends before finally
> > succeeding. ;-)
>
> The same question to you (I asked already Mose). What is your
> workflow, (I don't want to repeat my questions here to much
> crossposting)
I do as many local commits as necessary. If it takes longer to finish
my work, I "git pull --rebase" so that my commits stay on top. When I’m
done, I do "git rebase -i" and squash my commits into one. This is also
the time I do the ChangeLog messages for the commit message by
extracting them from the diff of my changes.
Bye,
Tassilo
- [AUCTeX-devel] patch with improved commit message, Uwe Brauer, 2016/09/17
- Re: [AUCTeX-devel] patch with improved commit message, Tassilo Horn, 2016/09/17
- Re: [AUCTeX-devel] patch with improved commit message, Uwe Brauer, 2016/09/17
- Re: [AUCTeX-devel] patch with improved commit message,
Tassilo Horn <=
- Re: [AUCTeX-devel] patch with improved commit message, Uwe Brauer, 2016/09/17
- Re: [AUCTeX-devel] patch with improved commit message, Tassilo Horn, 2016/09/18
- Re: [AUCTeX-devel] patch with improved commit message, Uwe Brauer, 2016/09/18
- Re: [AUCTeX-devel] patch with improved commit message, Tassilo Horn, 2016/09/18
- Re: [AUCTeX-devel] patch with improved commit message, Uwe Brauer, 2016/09/18
- Re: [AUCTeX-devel] patch with improved commit message, Tassilo Horn, 2016/09/19
- Re: [AUCTeX-devel] patch with improved commit message, Uwe Brauer, 2016/09/19