Message ID | 20241014-insane-ordering-v1-1-0b51976580ba@bootlin.com |
---|---|
State | New |
Headers | show |
Series | ref-manual: fix ordering of insane checks list | expand |
Hi Antonin, On 10/14/24 4:11 PM, Antonin Godard wrote: > It should be before "perllocalpod" and after "patch-status". > > Signed-off-by: Antonin Godard <antonin.godard@bootlin.com> > Reported-by: Quentin Schulz <quentin.schulz@cherry.de> Looks good to me, though not all items in the list are alphabetically sorted, still :) configure-gettext and staticdev are not in their right place, the rest looks fine I think. But progress being made, so: Reviewed-by: Quentin Schulz <quentin.schulz@cherry.de> Thanks! Quentin
diff --git a/documentation/ref-manual/classes.rst b/documentation/ref-manual/classes.rst index a715be97dc3469dc3c219b78548b79e8e3e4b95f..b92f4e4f20ea8f702c90f4e3d29251b2461d07d0 100644 --- a/documentation/ref-manual/classes.rst +++ b/documentation/ref-manual/classes.rst @@ -1464,6 +1464,9 @@ The tests you can list with the :term:`WARN_QA` and - ``patch-status:`` Checks that the ``Upstream-Status`` is specified and valid in the headers of patches for recipes. +- ``pep517-backend:`` checks that a recipe inheriting + :ref:`ref-classes-setuptools3` has a PEP517-compliant backend. + - ``perllocalpod:`` Checks for ``perllocal.pod`` being erroneously installed and packaged by a recipe. @@ -1478,9 +1481,6 @@ The tests you can list with the :term:`WARN_QA` and - ``perms:`` Currently, this check is unused but reserved. -- ``pep517-backend:`` checks that a recipe inheriting - :ref:`ref-classes-setuptools3` has a PEP517-compliant backend. - - ``pkgconfig:`` Checks ``.pc`` files for any :term:`TMPDIR`/:term:`WORKDIR` paths. Any ``.pc`` file containing these paths is incorrect since
It should be before "perllocalpod" and after "patch-status". Signed-off-by: Antonin Godard <antonin.godard@bootlin.com> Reported-by: Quentin Schulz <quentin.schulz@cherry.de> --- documentation/ref-manual/classes.rst | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) --- base-commit: 67a239f372ed1667c923e2f407da976627be2039 change-id: 20241014-insane-ordering-c327f3741ed4 Best regards,