[PATCH] config: adding dts tag for dts path patches

[PATCH] config: adding dts tag for dts path patches

[Manit Mahajan]

DTS provides a check-format script which runs a series of code quality
checks including formatting, linting, and type-checking using the
following tools: black, isort, pylama, and mypy. Developers are supposed
to run this script before submitting their patch series. In CI testing,
we want to run this formatting script on all patches which modify
dpdk/dts/* going forward, and submit a new patchwork check named
dts-check-format. In order to do this, we need to be able to set a tag
in the patch parser script for all patch series which meet this
condition. This series adds the dts tag and path assignment to
config/patch_parser.cfg.

Signed-off-by: Manit Mahajan
---
 config/patch_parser.cfg | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/config/patch_parser.cfg b/config/patch_parser.cfg
index 5757f9a..dac3b23 100644
--- a/config/patch_parser.cfg
+++ b/config/patch_parser.cfg
@@ -13,6 +13,7 @@ app = application
 license = documentation
 VERSION = documentation
 build = core
+dts = dts
 
 # This is an ordered list of the importance of each patch classification.
 # It should be used to determine which classification to use on tools which
@@ -22,4 +23,5 @@ priority_list =
     core,
     driver,
     application,
-    documentation
+    documentation,
+    dts
-- 
2.41.0

Re: [PATCH] config: adding dts tag for dts path patches

[Patrick Robb]

[-- Attachment #1: Type: text/plain, Size: 127 bytes --]

Reviewed-by: Patrick Robb <probb@iol.unh.edu>

Adding Luca since he is keen to see the dts format script run in CI
testing. :)

[-- Attachment #2: Type: text/html, Size: 217 bytes --]

Re: [PATCH] config: adding dts tag for dts path patches

[Aaron Conole]

Patrick Robb <probb@iol.unh.edu> writes:

> Reviewed-by: Patrick Robb <probb@iol.unh.edu>
>
> Adding Luca since he is keen to see the dts format script run in CI testing. :)

Should I hold off on Luca's ack/reviewed before applying?

Re: [PATCH] config: adding dts tag for dts path patches

[Patrick Robb]

[-- Attachment #1: Type: text/plain, Size: 583 bytes --]

No, I don't think that's necessary. What he described (run the format check
every time a file within /dts/* is being changed) will be accomplished with
this patch. And I don't think he has worked with the dpdk-ci repo
previously anyways.

On Fri, Oct 4, 2024 at 10:01 AM Aaron Conole <aconole@redhat.com> wrote:

> Patrick Robb <probb@iol.unh.edu> writes:
>
> > Reviewed-by: Patrick Robb <probb@iol.unh.edu>
> >
> > Adding Luca since he is keen to see the dts format script run in CI
> testing. :)
>
> Should I hold off on Luca's ack/reviewed before applying?
>
>

[-- Attachment #2: Type: text/html, Size: 1016 bytes --]