[LTP] [PATCH 1/1] doc: Sort items in the test catalog

[LTP] [PATCH 1/1] doc: Sort items in the test catalog

[Petr Vorel]

Sorting items with 'sort' in metadata/parse.sh does not help, because
whole path is sorted, not file name only.

NOTE: I would keep the 'sort' in metadata/parse.sh due JSON file build
reproducibility.

Signed-off-by: Petr Vorel <pvorel@suse.cz>
---
 doc/conf.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/doc/conf.py b/doc/conf.py
index 0bebd0e242..03fcff67e8 100644
--- a/doc/conf.py
+++ b/doc/conf.py
@@ -435,7 +435,7 @@ def generate_test_catalog(_):
 
     timeout_def = metadata['defaults']['timeout']
 
-    for test_name, conf in metadata['tests'].items():
+    for test_name, conf in sorted(metadata['tests'].items()):
         text.extend([
             f'{test_name}',
             len(test_name) * '-'
-- 
2.47.2


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] doc: Sort items in the test catalog

[Andrea Cervesato via ltp]

Hi!

On 2/12/25 15:24, Petr Vorel wrote:
> for test_name, conf in sorted(metadata['tests'].items()):

Obviously correct.

Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>


-- 
Mailing list info: https://lists.linux.it/listinfo/ltp

Re: [LTP] [PATCH 1/1] doc: Sort items in the test catalog

[Petr Vorel]

Hi Andrea,

> Hi!

> On 2/12/25 15:24, Petr Vorel wrote:
> > for test_name, conf in sorted(metadata['tests'].items()):

> Obviously correct.

> Reviewed-by: Andrea Cervesato <andrea.cervesato@suse.com>

Thanks for your review, merged.

Kind regards,
Petr

-- 
Mailing list info: https://lists.linux.it/listinfo/ltp