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.richard...@intel.com>

Acked-by: Chengwen Feng <fengcheng...@huawei.com>

Reply via email to