Lars-Dominik Braun <ldb@leibniz-psychology.org> writes:
Toggle quote (5 lines)
> Hi,>> the attached patch upgrades python-argcomplete to version 1.11.1 and includes a> fix from git for the testsuite and fish>=3.1.
Thanks! A few small nit-picks:
Toggle quote (5 lines)
> From f72dc474ff2e0ded162661b5c845371cb48c936d Mon Sep 17 00:00:00 2001> From: Lars-Dominik Braun <ldb@leibniz-psychology.org>> Date: Tue, 19 May 2020 09:24:03 +0200> Subject: [PATCH] gnu: python-argcomplete: Update to 1.11.1
Please use punctuations in commit messages.
Toggle quote (3 lines)
> * gnu/packages/python-xyz.scm (python-argcomplete)[version]: Upgrade> [arguments]: Remove, obsolete
Also mention the removed [inputs].
Toggle quote (3 lines)
> * gnu/packages/patches/python-argcomplete-1.11.1-fish31.patch: Add fix for fish 3.1> * gnu/local.mk (dist_patch_DATA): Add new file
Here it should say something along the lines of "...fish31.patch": New file....(dist_patch_DATA): Add it.
See the commit log for examples.
Toggle quote (16 lines)
> diff --git a/gnu/packages/patches/python-argcomplete-1.11.1-fish31.patch b/gnu/packages/patches/python-argcomplete-1.11.1-fish31.patch> new file mode 100644> index 0000000000..575c8abbb7> --- /dev/null> +++ b/gnu/packages/patches/python-argcomplete-1.11.1-fish31.patch> @@ -0,0 +1,35 @@> +From 08bfc8a788e8081515d733e67be026d051c726f7 Mon Sep 17 00:00:00 2001> +From: Evan <evanunderscore@gmail.com>> +Date: Thu, 19 Mar 2020 14:55:07 +1100> +Subject: [PATCH] Remove expected test failure for new versions of fish (#292)> +> +---> + test/test.py | 7 ++++++-> + 1 file changed, 6 insertions(+), 1 deletion(-)> +
Finally, can you remove this git patch header, and instead add a shortfree-style comment about what the patch does, and where it comes from(URL)? See other patches for examples.
Very minor issues, but getting these right will get your patches mergedfaster in the future. ;-)
Can you send an updated patch? TIA!