all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* Naming scheme for branches which will not be merged.
@ 2014-12-11 18:29 Artur Malabarba
  2014-12-11 19:29 ` Stefan Monnier
  0 siblings, 1 reply; 3+ messages in thread
From: Artur Malabarba @ 2014-12-11 18:29 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

2014-12-11 17:21 GMT+00:00 Stefan Monnier >
> Maybe we should decide of a particular branch-naming scheme which states
> clearly that the branch won't be merged into master (it will be
> applied as a brand new patch with a brand new commit message).

I agree with Stefan on this.

I find that the most convenient way of working on something while
getting feedback is to push it to a branch in origin. For this
specific purpose a branch which is not meant to be merged has a few
advantages over “real” branches.

We can develop without worrying about the ChangeLog file.
Each commit doesn't necessarily have to be a full change, which
naturally lends itself to the workflow of this list where diffs
commonly incite feedback which leads to small changes.
Commit messages don't have to follow any convention. If you make a
short commit to fix a typo, you can just say “Typo” and move on.
However, they should still be useful messages!

Once this branch is given as finished, the creator (or anyone else,
really), turns into one (or several) proper commit(s). This is when we
write real changelog/commit messages. How this is done is irrelevant,
you can use rebase+squash+merge or manually apply a patch on master.

The two questions I can think of are

What naming scheme should we use? Either prefix the branch name with
something really obvious, like “dont-merge/BRANCH-NAME”, so that
people who unaware of this convention don't make that mistake. Or
prefix it with any of the usual words that denote in-development, such
as “dev/BRANCH-NAME”.
Should the creator of the branch add his/her username to it? i.e.
“dont-merge/malabarba/BRANCH-NAME”.



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Naming scheme for branches which will not be merged.
  2014-12-11 18:29 Naming scheme for branches which will not be merged Artur Malabarba
@ 2014-12-11 19:29 ` Stefan Monnier
  2014-12-11 23:13   ` Artur Malabarba
  0 siblings, 1 reply; 3+ messages in thread
From: Stefan Monnier @ 2014-12-11 19:29 UTC (permalink / raw)
  To: Artur Malabarba; +Cc: emacs-devel

> What naming scheme should we use? Either prefix the branch name with
> something really obvious, like “dont-merge/BRANCH-NAME”, so that
> people who unaware of this convention don't make that mistake. Or
> prefix it with any of the usual words that denote in-development, such
> as “dev/BRANCH-NAME”.

I vote against `dont-merge', and propose `scratch' as another contender.

> Should the creator of the branch add his/her username to it? i.e.
> “dont-merge/malabarba/BRANCH-NAME”.

I don't see a strong need to put the user in the branch name, unless you
want to state explicitly that other developers shouldn't mess with it.


        Stefan



^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: Naming scheme for branches which will not be merged.
  2014-12-11 19:29 ` Stefan Monnier
@ 2014-12-11 23:13   ` Artur Malabarba
  0 siblings, 0 replies; 3+ messages in thread
From: Artur Malabarba @ 2014-12-11 23:13 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

[-- Attachment #1: Type: text/plain, Size: 502 bytes --]

On 11 Dec 2014 17:29, "Stefan Monnier" <monnier@iro.umontreal.ca> wrote:
>
> > What naming scheme should we use? Either prefix the branch name with
> > something really obvious, like “dont-merge/BRANCH-NAME”, so that
> > people who unaware of this convention don't make that mistake. Or
> > prefix it with any of the usual words that denote in-development, such
> > as “dev/BRANCH-NAME”.
>
> I vote against `dont-merge', and propose `scratch' as another contender.

Oh, I like that!

[-- Attachment #2: Type: text/html, Size: 680 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2014-12-11 23:13 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-12-11 18:29 Naming scheme for branches which will not be merged Artur Malabarba
2014-12-11 19:29 ` Stefan Monnier
2014-12-11 23:13   ` Artur Malabarba

Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.