-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Prepare release 1.27.0 #3306
Prepare release 1.27.0 #3306
Conversation
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
There is one commit in the UI https://github.com/jaegertracing/jaeger-ui/commits/master. I am not sure if it is worth releasing and including it here. |
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 think you want to do the UI release first and include the new hash in this PR so that we don't have multiple release-related commits
This looks like a new feature, so warrants a new UI release imo. |
Signed-off-by: Joe Elliott <number101010@gmail.com>
Signed-off-by: Joe Elliott <number101010@gmail.com>
Codecov Report
@@ Coverage Diff @@
## master #3306 +/- ##
=======================================
Coverage 95.98% 95.98%
=======================================
Files 259 259
Lines 15434 15434
=======================================
Hits 14814 14814
- Misses 525 526 +1
+ Partials 95 94 -1
Continue to review full report at Codecov.
|
Short description of the changes
make changelog
that only default token permissions are needed