[dpdk-dev] [PATCH v2] devtools: clarify that lines up to 100 characters are ok

fengchengwen fengchengwen at huawei.com
Tue Oct 26 02:30:24 CEST 2021


On 2021/10/20 22:26, Bruce Richardson wrote:
> 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>

Acked-by: Chengwen Feng <fengchengwen at huawei.com>



More information about the dev mailing list