-
Notifications
You must be signed in to change notification settings - Fork 452
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
release(v0.19): add initial change log #965
Conversation
Codecov ReportPatch and project coverage have no change.
Additional details and impacted files@@ Coverage Diff @@
## main #965 +/- ##
=====================================
Coverage 69.9% 69.9%
=====================================
Files 116 116
Lines 9027 9027
=====================================
+ Hits 6310 6315 +5
+ Misses 2717 2712 -5 see 2 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
I think we should deal with #969 before releasing. Is it intentional that the integration tests job is getting skipped in CI? Should we run it manually before release? |
Yeah I intend to expand the integration test coverage so I expect the time to run it grows over time. And it may not needed to run for every commit/PR. But we should run it before release. |
Listed the following issues as blocking before 0.19 release, feel free to add more:
|
@TommyCpp Is the blocker still the memory issue? |
Should we place a time limit on the #901 change, if it's not merged by X date then we merge and release? |
I think that's all changes we want to land in v0.19. PTAL cc @open-telemetry/rust-approvers |
🚀 thank you @TommyCpp |
We are still blocked by some memory issue investigation but once it resolved we should be able to cut a release