[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Lack of tooling slowing down contributions
From: |
Damien Cassou |
Subject: |
Lack of tooling slowing down contributions |
Date: |
Fri, 14 Jun 2019 09:38:00 +0200 |
Hi,
I've submitted several patches to Emacs. Because I don't do it very
often, I tend to forget the details in CONTRIBUTE. Because I'm lazy,
I'm not reading CONTRIBUTE before each new contribution. As a result,
my patches are not good enough and reviewers have to spend time telling
me how to do things. I don't think reviewers should spend their
valuable time checking for details that can be checked by a machine.
These checks could be done either (by order of preference):
1. on the contributor's Emacs instance while s·he writes (e.g., flymake
on .texi files and commit messages);
2. on the contributor's system before s·he sends the patches (e.g.,
./check_patches.sh *.patch);
3. on a continuous integration server that would automatically answer by
email on thread with patches, listing the problems.
Here is a list of checks that could probably be automatized, saving time
for reviewers and contributors:
- the commit message format (e.g., dots after every sentence,
double-space after dots, no indentation, line length, the
Copyright-paperwork-exempt token, every single change is documented,
no colon if another function of the same file has the same comment…)
- NEWS format (e.g., there is always either a '+++' or a '---', the
first sentence stops at the first line, double space after dots, …)
- the .texi format (e.g., double space after dots…)
- the elisp format (e.g., a :version property on new defcustoms, all
arguments are listed on the first docstring line…)
It would also be nice that tooling within Emacs helps contributors get
their contributions right. For example, I would expect M-q to add the
double-spacing at the end of sentences.
BTW, can you please C.C. me if you answer this message because I'm not
subscribed to the mailing list?
Thanks for reading and sorry for the length of this email.
--
Damien Cassou
http://damiencassou.seasidehosting.st
"Success is the ability to go from one failure to another without
losing enthusiasm." --Winston Churchill
- Lack of tooling slowing down contributions,
Damien Cassou <=
- Re: Lack of tooling slowing down contributions, Eli Zaretskii, 2019/06/14
- Re: Lack of tooling slowing down contributions, Noam Postavsky, 2019/06/14
- Re: Lack of tooling slowing down contributions, Juri Linkov, 2019/06/16
- Re: Lack of tooling slowing down contributions, Phil Sainty, 2019/06/16
- Re: Lack of tooling slowing down contributions, Van L, 2019/06/17
- Re: Lack of tooling slowing down contributions, Marcin Borkowski, 2019/06/17
- Re: Lack of tooling slowing down contributions, Yuri Khan, 2019/06/17
- Re: Lack of tooling slowing down contributions, Juri Linkov, 2019/06/17