Skip to content

Fix path for script#218

Merged
isabelrios merged 11 commits intomainfrom
irios-fix-path-testrail-scan-script
Feb 4, 2026
Merged

Fix path for script#218
isabelrios merged 11 commits intomainfrom
irios-fix-path-testrail-scan-script

Conversation

@isabelrios
Copy link
Copy Markdown
Collaborator

My bad.. did a copy paste from the action in firefox ios repo.. we need to fix the path to the script

@AaronMT
Copy link
Copy Markdown
Collaborator

AaronMT commented Feb 2, 2026

@isabelrios Did you need a review?

@isabelrios
Copy link
Copy Markdown
Collaborator Author

@isabelrios Did you need a review?

Oh sorry not yet, fixing the slack payload to be sent once the path is correct

@AaronMT
Copy link
Copy Markdown
Collaborator

AaronMT commented Feb 4, 2026

Looks OK to me.

Side question, for report generation however, would this better served as an HTML report instead of a Slack notification?

@isabelrios
Copy link
Copy Markdown
Collaborator Author

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!

@isabelrios isabelrios merged commit 5bd9d13 into main Feb 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants