Martchus
Martchus
You need to run tidy and maybe adapt/extend tests. Otherwise the PR looks good.
Looks like tests definitely need to be adapted. If you need help with that I can try to help you - especially because this issue is probably quite important right...
Looks like I cannot push to this branch. Can you cherry-pick https://github.com/os-autoinst/os-autoinst/commit/dce7697a6468a34ab122933465ade8bfb7a7740c so we can merge this PR?
Because otherwise the `codecov/project/fully_covered` check will fail on every PR. Of course we can also block this feature on having proper tests covering both cases always. Or we remove the...
Then I suggest you drop my commit again. The target can be reduced by modifying `codecov.yml`. The tests could also be extended by mocking the QEMU version and running relevant...
But it looks like style tests are failing so you should probably invoke the test `test-local-perl-style` locally and see what's wrong.
I'm not using Fedora myself so maybe the packages are broken. However, before we jump to conclusions: Did you check that you have added the correct repository - the one...
The way you want to configure it is basically how most people (including me) configure it so this should normally be possible and work. Note that there's no setting to...
Supposedly in your case a surplus empty/disabled entry for the Plasmoid is shown as well. This definitely doesn't help to make it less confusing. (See https://github.com/Martchus/syncthingtray/issues/239 for this issue.) If...
Maybe a libc++ or configuration specific code. Otherwise I fail to see why this can compile anywhere. I can have a look later but since you can already reproduce it...