emacs-orgmode
[Top][All Lists]
Advanced

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

Re: [PATCH] implementing `with' as a list, and respecting `deps' order.


From: Mario Frasca
Subject: Re: [PATCH] implementing `with' as a list, and respecting `deps' order.
Date: Wed, 3 Jun 2020 09:46:10 -0500
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.1

comments on the html page org-contribute.html:

is there any mention of git-send-mail in the org-contribute guide?  I don't see it.  nor do I find hints on what to do once you have produced files by git-format-patch.  your "send as attachment" would be useful there.  or a mention of a (unix) command-line sequence, including how to follow-up to previous messages.

the procedure mentioned there guides me in creating several patch files, one per commit.  it does not describe the "squashing" you suggested me.  I'm following your guide, but you might want to review the page yourself.

the `#commit-messages' section is very clear, I had totally missed it.  sorry.

I'm reviewing the commit message accordingly, and will reply to my initial message, attaching the new patch.

ciao, MF

On 02/06/2020 22:38, Kyle Meyer wrote:
Mario Frasca writes:

  From 436bfd0b9fd656f52ea9d7e6a6a665a32564ae93 Mon Sep 17 00:00:00 2001
From: Mario Frasca <mario@anche.no>
Date: Tue, 2 Jun 2020 15:46:20 +0000
Subject: [PATCH] implementing `with' as a list, and respecting `deps' order.
Thank you for sending an updated patch.  Unfortunately it looks like it
got mangled by your email client and can't be understood by `git am'.
This list is okay with patches being sent as attachments, and going that
route gets around needing to re-configure your email client or use
git-send-email.

Before sending the updated patch, please revise the commit message to
more closely match the conventions described at
<https://orgmode.org/worg/org-contribute.html>.  Looking over some
recent commits in the repo should give you a good sense for the expected
changelog-like entries as well other style aspects (e.g., it is common
to start the subject with "<file|area>: " like "org-plot: ..."), which
gives log readers a sense for the general topic affected by the commit.

Please send the next patch as a follow-up to the original thread to keep
the discussion in one place.

Thanks again.



reply via email to

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