-
Notifications
You must be signed in to change notification settings - Fork 196
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
rfc: cloud snapshot tests #6044
Conversation
An RFC for #5958. A proposal for a feature that will capture a snapshot of the synthesis output from `wing test` to reduce regressions in cloud implementations.
Thanks for opening this pull request! 🎉
|
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.
There probably needs to be more consideration intertwined with things like #5576. When we move forward on that we should take this snapshotting into account.
I think there's enough here to move forward though. Especially since the hard part of this is the snapshot implementation itself which probably won't care much about how/where the lines are drawn for what a test
is.
Thanks for contributing, @eladb! This PR will now be added to the merge queue, or immediately merged if |
Congrats! 🚀 This was released in Wing 0.67.5. |
An RFC for #5958.
A proposal for a feature that will capture a snapshot of the synthesis output from
wing test
to reduce regressions in cloud implementations.By submitting this pull request, I confirm that my contribution is made under the terms of the Wing Cloud Contribution License.