-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
cloud: New output v2 #3072
cloud: New output v2 #3072
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3072 +/- ##
==========================================
+ Coverage 75.29% 75.31% +0.01%
==========================================
Files 232 233 +1
Lines 17593 17607 +14
==========================================
+ Hits 13247 13260 +13
- Misses 3494 3495 +1
Partials 852 852
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.
I know this doesn't have any of the logic, but just wanted to 👍 the general versioning approach
@codebien Is this PR ready for review? I've been ignoring it since it's marked as draft. |
@imiric not really, the attempt here is to collect all the other PRs and then do a single merge to master. Of course, without really reviewing it because we reviewed the smaller PRs. |
If we are going to get this merged before all the other get merged, maybe drop the |
Description updated |
It adds the foundation for a new cloud output versioned as v2.
It depends on the following PRs as a prerequisite: