Conversation
|
@isabelrios Did you need a review? |
Oh sorry not yet, fixing the slack payload to be sent once the path is correct |
|
Looks OK to me. Side question, for report generation however, would this better served as an HTML report instead of a Slack notification? |
Ideally we will not have that long list once we do the first path to add the testrail ids, so the slack will only show a few which may be nicer then... we also have the full list in the artifacts, that is accessible from the slack notification: https://github.com/mozilla-mobile/testops-tools/actions/runs/21626293492/attempts/1#summary-62327210175 we can start with this and create a html report if that is easier to track / see. Thanks! |
My bad.. did a copy paste from the action in firefox ios repo.. we need to fix the path to the script