[dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok
Xia, Chenbo
chenbo.xia at intel.com
Fri Oct 22 10:21:09 CEST 2021
> -----Original Message-----
> From: dev <dev-bounces at dpdk.org> On Behalf Of Bruce Richardson
> Sent: Wednesday, October 20, 2021 10:26 PM
> 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>
> ---
> 2.32.0
Glad to see this :)
Acked-by: Chenbo Xia <chenbo.xia at intel.com>
More information about the dev
mailing list