mbox series

[kvm-unit-tests,0/6] Checking and CI improvements

Message ID 20240726070456.467533-1-npiggin@gmail.com (mailing list archive)
Headers show
Series Checking and CI improvements | expand

Message

Nicholas Piggin July 26, 2024, 7:04 a.m. UTC 
Here's some assorted fixes and improvements to static checking
and CI.

checkpatch support is a big one and you could consider it RFC
for now. We may want to customise it somewhat (but IMO fewer
customisations the easier to maintain the script if we can agree
on close to kernel style). It's nice because it gets DCO, license
headers, etc., not just code style.

Here is a pipeline with this series applied:

https://gitlab.com/npiggin/kvm-unit-tests/-/pipelines/1388888286

Thanks,
Nick

Nicholas Piggin (6):
  gitlab-ci: fix CentOS mirror list
  arm: Fix kerneldoc
  gitlab-ci: upgrade to CentOS 8
  gitlab-ci: Move check-kerneldoc test out of the centos test
  gitlab-ci: add a shellcheck test
  checkpatch support

 .gitlab-ci.yml         |   83 +-
 arm/fpu.c              |   24 +-
 scripts/check-patch.py |   61 +
 scripts/checkpatch.pl  | 7839 ++++++++++++++++++++++++++++++++++++++++
 4 files changed, 7986 insertions(+), 21 deletions(-)
 create mode 100755 scripts/check-patch.py
 create mode 100755 scripts/checkpatch.pl

Comments

Andrew Jones July 29, 2024, 1:12 p.m. UTC | #1 
On Fri, Jul 26, 2024 at 05:04:41PM GMT, Nicholas Piggin wrote:
> Here's some assorted fixes and improvements to static checking
> and CI.
> 
> checkpatch support is a big one and you could consider it RFC
> for now. We may want to customise it somewhat (but IMO fewer
> customisations the easier to maintain the script if we can agree
> on close to kernel style). It's nice because it gets DCO, license
> headers, etc., not just code style.
> 
> Here is a pipeline with this series applied:
> 
> https://gitlab.com/npiggin/kvm-unit-tests/-/pipelines/1388888286
> 
> Thanks,
> Nick
> 
> Nicholas Piggin (6):
>   gitlab-ci: fix CentOS mirror list
>   arm: Fix kerneldoc
>   gitlab-ci: upgrade to CentOS 8
>   gitlab-ci: Move check-kerneldoc test out of the centos test
>   gitlab-ci: add a shellcheck test
>   checkpatch support
> 
>  .gitlab-ci.yml         |   83 +-
>  arm/fpu.c              |   24 +-
>  scripts/check-patch.py |   61 +
>  scripts/checkpatch.pl  | 7839 ++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 7986 insertions(+), 21 deletions(-)
>  create mode 100755 scripts/check-patch.py
>  create mode 100755 scripts/checkpatch.pl
> 
> -- 
> 2.45.2
>

For the series,

Acked-by: Andrew Jones <andrew.jones@linux.dev>

Thanks,
drew
Andrew Jones Aug. 12, 2024, 2:32 p.m. UTC | #2 
On Fri, Jul 26, 2024 at 05:04:41PM GMT, Nicholas Piggin wrote:
> Here's some assorted fixes and improvements to static checking
> and CI.
> 
> checkpatch support is a big one and you could consider it RFC
> for now. We may want to customise it somewhat (but IMO fewer
> customisations the easier to maintain the script if we can agree
> on close to kernel style). It's nice because it gets DCO, license
> headers, etc., not just code style.
> 
> Here is a pipeline with this series applied:
> 
> https://gitlab.com/npiggin/kvm-unit-tests/-/pipelines/1388888286
> 
> Thanks,
> Nick
> 
> Nicholas Piggin (6):
>   gitlab-ci: fix CentOS mirror list
>   arm: Fix kerneldoc
>   gitlab-ci: upgrade to CentOS 8
>   gitlab-ci: Move check-kerneldoc test out of the centos test
>   gitlab-ci: add a shellcheck test
>   checkpatch support
> 
>  .gitlab-ci.yml         |   83 +-
>  arm/fpu.c              |   24 +-
>  scripts/check-patch.py |   61 +
>  scripts/checkpatch.pl  | 7839 ++++++++++++++++++++++++++++++++++++++++
>  4 files changed, 7986 insertions(+), 21 deletions(-)
>  create mode 100755 scripts/check-patch.py
>  create mode 100755 scripts/checkpatch.pl
> 
> -- 
> 2.45.2
>

Merged

Thanks,
drew