-
Notifications
You must be signed in to change notification settings - Fork 896
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 v1.7.0 #1926
Release v1.7.0 #1926
Conversation
Looking at the changelog I don't see any changes to the schema so the schema file for 1.7.0 needs just an empty section for 1.7.0 version. @carlosalberto this should do it (put it in schemas/1.7.0):
|
Need action on #1658 before releasing I think @Oberon00 @bogdandrutu |
Yes, please can we resolve #1658 now? It has been hanging around for quite a while and would be cumbersome to implement it if we don't release it in the same release that added the InstrumentationLibrary to the sampler. |
I believe we are using the catch-the-train release model so #1658 is better merged soon or it is not going to make it (or do we delay the release since it is critical?) |
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, thanks @carlosalberto!
Hey, was wondering if we should postpone this release to the first days of October, so related issues are properly solved? Or would that hurt anybody? |
This PR was marked stale due to lack of activity. It will be closed in 7 days. |
#1658 is closed. Are we waiting for something else? |
Let's try to wait for #1970 by EOD, and then we finally do our release ;) |
@carlosalberto I will be travelling and may not be able to merge later today, please do it if you can. |
September release, following our monthly cadence.
@reyang Are we ready to go, metrics-wise?
@bogdandrutu @dyladan Should we include #1923 in this release?
cc @tigrannajaryan