New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow passing --file-reporter option to test running refs #69425 #120716
allow passing --file-reporter option to test running refs #69425 #120716
Conversation
f1a0b0d
to
9bfbd1c
Compare
For some reason, the |
@Jasguerrero can you review this one? |
9bfbd1c
to
0e1bae7
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
auto label is removed for flutter/flutter, pr: 120716, due to - Please get at least one approved review if you are already a member or two member reviews if you are not a member before re-applying this label. Reviewers: If you left a comment approving, please use the "approve" review action instead. |
auto label is removed for flutter/flutter, pr: 120716, due to Validations Fail. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Allow passing
--file-reporter
option to test runner so that both machine and human readable output can be observed in a CI setting.Supersedes #111969.
Fixes #69425.
Note: I did not made the initial commit. It was made entirely by @daniel-v, so all credits goes to him. I rebased his patch and I'm going to continue the PR process by fixing tests/checks and reacting to reviewers.
Pre-launch Checklist
///
).