-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Prepare E2E tooling for better message passing #15843
Conversation
Codecov Report
Flags with carried forward coverage won't be shown. Click here to find out more. |
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.
Could you please provide some context for this wider refactoring? Just a link to a ticket in the PR description would be great.
There is no associated GitHub issue nor internal architecture link |
Basically I'm switching the communication mechanism between the CLI and tests to be files rather than formatted stdout so we can now show what's happening during setup |
Motivation
Large refactor is incoming and this is the first step