[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#37444] [PATCH] added aerc, a simple email client and its dependcies

From: Ludovic Courtès
Subject: [bug#37444] [PATCH] added aerc, a simple email client and its dependcies
Date: Thu, 26 Sep 2019 11:37:45 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.3 (gnu/linux)

Hi Martin,

Martin Becze <address@hidden> skribis:

> On 2019-09-18 09:01, Ricardo Wurmus wrote:
>> Hi Martin,
>>> This adds aerc, which is a terminal based email client written in go.
>> Thank you for your patch!
>> Could you please separate the independent changes into separate commits
>> and add commit summaries?  For example, the update of
>> go-github-com-mitchellh-go-homedir should be done separately.
>> Please also explain why “inputs” had to be changed to
>> “propagated-inputs” in some packages, and mention these kind of changes
>> in the commit summary.
>> Thanks!
> just bumping, updated patch has been made and split into commits now.
> let me know if that is ok

Like Ricardo mentioned, it would be great if patch #1 were split into
several commits, one per new package.  That’s how we usually do it.

I saw your commit logs provide explanations.  However, they are all on
the “subject line” of the commit log.  Could you take a look at
regarding the format of commit logs, or run, say, “git log
gnu/packages/golang.scm” to view examples?

I could do these changes on your behalf, but I’d rather let you do it to
make sure I don’t mess up or misrepresent what you did!


reply via email to

[Prev in Thread] Current Thread [Next in Thread]