action #97403
closedcoordination #91646: [saga][epic] SUSE Maintenance QA workflows with fully automated testing, approval and release
openQA Project (public) - coordination #91467: [epic] Surface openQA failures per squad in a single place
openqa-review: Polish job group section titles in todo-only mode size:S
0%
Description
Motivation¶
Acceptance criteria¶
- AC1: job groups in https://openqa.io.suse.de/openqa-review/openqa_suse_de_todo_only.html are not listed as list entries but true section headers
Suggestions¶
- Understand how https://github.com/os-autoinst/openqa_review/pull/171 introduced the single-line todo-only mode and try to handle job group names better
- See
for one suggestion
Updated by okurz over 3 years ago
- Subject changed from openqa-review: Polish to openqa-review: Polish job group section titles in todo-only mode
Updated by okurz over 3 years ago
- Subject changed from openqa-review: Polish job group section titles in todo-only mode to openqa-review: Polish job group section titles in todo-only mode size:S
- Status changed from New to Workable
Updated by mkittler over 3 years ago
- Status changed from Workable to In Progress
If you create a PR you can reference it like this:
Updated by kodymo over 3 years ago
Changes are merged in Git, waiting for deployment.
Updated by okurz over 3 years ago
- Status changed from Resolved to Feedback
hi, have you looked at https://openqa.io.suse.de/openqa-review/openqa_suse_de_todo_only.html ? It looks much better but there is a minor flaw in the first section header which is not indented the same way as the following. I suggest to indent both the same way, either indent the first one or ensure that the others are not indented
Updated by okurz over 3 years ago
- Status changed from Feedback to Resolved
The follow-up PR was https://github.com/os-autoinst/openqa_review/pull/179 which is in production and looks just fine. Thank you! Next time you can happily resolve the ticket yourself as soon as such requirements are met