Closed Vogtinator closed 1 year ago
Unfortunately TTM itself doesn't quite fit here due to the reasons outlined in the code and the additional checks are not a good fit for adding it into the go pipeline description, so write yet another OBS publishing bot!
I'm also trying to assess how well this would fit into the existing qac container-release-bot code. It's unfortunately different from containers in many places, which might end up making the code more complex than maintaining a separate bot altogether.
For now doing it like this appeared to be the easiest and quickest option and should still be rather maintainable due to its simplicity.
Patch coverage has no change and project coverage change: +14.87
:tada:
Comparison is base (
8b1c308
) 13.58% compared to head (e9acafe
) 28.45%.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
LGTM. some small nits inline. I'd +1 it
Should be addressed now
Unfortunately TTM itself doesn't quite fit here due to the reasons outlined in the code and the additional checks are not a good fit for adding it into the go pipeline description, so write yet another OBS publishing bot!
For the initial test runs this is not triggered by timer runs yet.
CC @fcrozat