Git Protocol

Eugenio the CD branch for example had commits in excess of 100. The last
octopus video pull requests likewise. Intermingled among those are probably
a few bugs that is much more difficult to understand after everything is
cramped together.


On Thu, Jul 17, 2014 at 1:57 PM, Eugenio Depalo <notifications@github.com>
wrote:

> In protocols/git/README.md:
>
> > +
> > +When you've staged the changes, commit them.
> > +
> > +    git status
> > +    git commit --verbose
> > +
> > +Write a [good commit message]. Example format:
> > +
> > +    Present-tense summary under 50 characters
> > +
> > +    * More information about commit (under 72 characters).
> > +    * More information about commit (under 72 characters).
> > +
> > +    http:://project.management-system.com/ticket/123
> > +
> > +If you've created more than one commit, use a rebase to squash them into
>
> I believe in the context of reading master history, the only thing that
> matters is the PR changes in their entirety. I cannot really imagine a case
> where I'd like to see intermediary states of a person's feature branch to
> extract some meaningful informations out of it.
>
> I also commit typos quite often and I believe it's not really informative
> for anyone to have those as separate commit messages. I would usually just
> amend or rebase to avoid having history interrupted by trivial commits.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/alphasights/guides/pull/4/files#r15052657>.
>



-- 
*Dee Zsombor*

We are hiring <http://www.alphasights.com/careers/overview>

alphasights.com <http://www.alphasights.com/>  |  Knowledge brokers for the
information age.
58 Davies Street, London, W1K 5JF, United Kingdom
Tel   |  Mob

This e-mail message is intended only for the recipient identified in the
address of this message. It may contain information that is confidential,
privileged or subject to legal restrictions and penalties regarding its
unauthorized disclosure and use. Any unauthorized review, copying,
disclosure, use or distribution is prohibited. If you have received this
e-mail message in error, please notify the sender immediately by reply
e-mail and delete this message, and any attachments, from your system.
AlphaSights Ltd is a Private Limited Company registered in England No.
6534480.