[dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok
Walsh, Conor
conor.walsh at intel.com
Thu Oct 28 15:14:53 CEST 2021
> From: dev <dev-bounces at dpdk.org> On Behalf Of Bruce Richardson
> Sent: Wednesday 20 October 2021 15:26
> To: dev at dpdk.org
> Cc: Andrew Rybchenko <andrew.rybchenko at oktetlabs.ru>; Richardson,
> Bruce <bruce.richardson at intel.com>
> Subject: [dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100
> characters are ok
>
> Since we allow line lengths of up to 100, and the CI checkpatches job
> only check for that amount, the rest of our tooling and docs should
> reflect this reality. Therefore we can:
>
> * adjust the editorconfig to use that value, to save editors (e.g. vim)
> from automatically wrapping lines at 80 characters when typing.
> [Since python checkers all seem to expect 79 character lines max, add
> for python only a 79-char max line length.]
>
> * change the default line length setting in checkpatches script to 100
> so as it matches CI and pre-merge checks.
>
> * update the docs to clarify that while 80 chars is recommended, up to
> 100 characters is acceptable.
>
> Signed-off-by: Bruce Richardson <bruce.richardson at intel.com>
> ---
I think it would be great to have clarity around this, since it seems to already be allowed.
Acked-by: Conor Walsh <conor.walsh at intel.com>
More information about the dev
mailing list