-
Notifications
You must be signed in to change notification settings - Fork 306
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
[repo] Add dedicated CI for persistent storage projects #1397
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1397 +/- ##
==========================================
- Coverage 73.91% 73.81% -0.11%
==========================================
Files 267 261 -6
Lines 9615 9608 -7
==========================================
- Hits 7107 7092 -15
- Misses 2508 2516 +8
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.
LGTM, I am not sure if we always need to release both oaclages together, but if you think so, we can ,merge as is.
@Kielek We don't technically have to release them together, but I think doing so simplifies things and reduces the chance of any issues. These packages are very similar to the I think the general wisdom to be gleamed from runtime team here is that it is better to release the whole graph together instead of trying to micro-manage the versions. We could always make it more granular if a strong need arises. |
Changes