Closed fernflower closed 3 years ago
Can one of the admins verify this patch?
Wouldn't it make more sense to have these as an environment variable which we can set in the project configuration on GitHub?
@vinzenz yeah, definitely. Will do tomorrow, thanks!
Good news - with this compose the tests are fine https://api.dev.testing-farm.io/v0.1/requests/4d91e6bf-6261-4a7c-b9d3-9b1869f66f9b
Thank you for contributing to the Leapp project!
Please note that every PR needs to comply with the Leapp Guidelines and must pass all tests in order to be mergable. If you want to re-run tests or request review, you can use following commands as a comment:
Please open ticket in case you experience technical problem with the CI. (RH internal only)
Note: In case there are problems with tests not being triggered automatically on new PR/commit or pending for a long time, please consider rerunning the CI by commenting leapp-ci build (might require several comments). If the problem persists, contact leapp-infra.