diff mbox series

ref-manual: fix ordering of insane checks list

Message ID 20241014-insane-ordering-v1-1-0b51976580ba@bootlin.com
State New
Headers show
Series ref-manual: fix ordering of insane checks list | expand

Commit Message

Antonin Godard Oct. 14, 2024, 2:11 p.m. UTC
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,

Comments

Quentin Schulz Oct. 15, 2024, 11:26 a.m. UTC | #1
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 mbox series

Patch

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