Yiannis

Results 29 comments of Yiannis

> Sorry, but I don't think this is a good approach of merging things automatically, even with 2 approvals. And I'm afraid it may bring more troubles then benefits to...

FYI I cant open the link

for the sake of completeness and avoid any confusion I fixed the issue with the test and some requests from feedback. But I flag the PR as `invalid`

> > Please provide a commit message. from the reviewers perspective I would like to know the context of the changes. for instance: in what logic or necessity there is...

![overview_modal](https://github.com/user-attachments/assets/25e9b959-0756-46d4-be8f-bc79f50483d7) ![overview_modal_submitted](https://github.com/user-attachments/assets/943f9e4b-0aa6-4e3d-a9f2-1956f4ed00af)

> ``` > fa-comment > ``` I will take a look on 3 tmr

> please describe the relation to > > * [Add option to restart jobs upon comment submission #5946](https://github.com/os-autoinst/openQA/pull/5946) The difference between this PR and the previews is that the old...

UPDATED screenshot ** ![image](https://github.com/user-attachments/assets/041bda5c-95a6-450a-8526-dd7799905236) **

I believe I addressed everything except the comments about the `and` substitution of `or` for the restart button

Some comments on 717e152b9f66615a5752397814660c9c94024d22 - Review commit message for main changes - not sure if we prefer to rename the restartJob in openqa.js - not sure if we prefer to...