-
Notifications
You must be signed in to change notification settings - Fork 470
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
Preparing release 1.8.0 #543
Conversation
CHANGELOG.md
Outdated
- **General:** bump url-parse version to 1.4.7 ([#542](https://github.com/jaegertracing/jaeger-ui/pull/542), [@rubenvp8510](https://github.com/rubenvp8510)) | ||
- **Search** Fix span links for leading 0s trace ID ([#539](https://github.com/jaegertracing/jaeger-ui/pull/539), [@everett980](https://github.com/everett980)) | ||
- **General:** Reorder, rename, and fix "About Jaeger" links ([#540](https://github.com/jaegertracing/jaeger-ui/pull/540), [@yurishkuro](https://github.com/yurishkuro)) | ||
- **Search:** Fix search.maxLimit configuration ([#533](https://github.com/jaegertracing/jaeger-ui/pull/533), [@rubenvp8510](https://github.com/rubenvp8510)) |
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.
There are two spaces between the title and the PR link
- **Trace detail:** Fix trace scoped links not supporting numeric fields ([#538](https://github.com/jaegertracing/jaeger-ui/pull/538), [@william-tran](https://github.com/william-tran)) | ||
- **Search:** Search results link spans ([#536](https://github.com/jaegertracing/jaeger-ui/pull/536), [@everett980](https://github.com/everett980)) | ||
|
||
|
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.
Remove one of the empty lines.
Codecov Report
@@ Coverage Diff @@
## master #543 +/- ##
==========================================
- Coverage 92.98% 92.94% -0.05%
==========================================
Files 197 197
Lines 4848 4848
Branches 1177 1177
==========================================
- Hits 4508 4506 -2
- Misses 300 302 +2
Partials 40 40
Continue to review full report at Codecov.
|
CHANGELOG.md
Outdated
|
||
- **Trace detail:** Fix trace scoped links not supporting numeric fields ([#538](https://github.com/jaegertracing/jaeger-ui/pull/538), [@william-tran](https://github.com/william-tran)) | ||
|
||
- **Search:** Search results link spans ([#536](https://github.com/jaegertracing/jaeger-ui/pull/536), [@everett980](https://github.com/everett980)) |
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.
this is a new feature, I believe, and because of that @everett980 mentioned offline that we may want to do a minor release, not a patch.
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.
Alright I will move it and change the version
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
3f4de11
to
443939b
Compare
CHANGELOG.md
Outdated
@@ -1,5 +1,20 @@ | |||
# Releases | |||
|
|||
## v1.7.1 (March 12, 2020) |
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.
1.8.0
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
if I am not mistaken, prettier doesn't like something in the changelog
|
I think the linter is complaining because of this: Check the double space between configuration and the first parenthesis.
|
Signed-off-by: Pavol Loffay <ploffay@redhat.com>
* Preparing release 1.7.1 Signed-off-by: Pavol Loffay <ploffay@redhat.com> * Update changelog Signed-off-by: Pavol Loffay <ploffay@redhat.com> * Update version Signed-off-by: Pavol Loffay <ploffay@redhat.com> * Fix verson Signed-off-by: Pavol Loffay <ploffay@redhat.com> * Remove double space Signed-off-by: Pavol Loffay <ploffay@redhat.com> Signed-off-by: vvvprabhakar <vvvprabhakar@gmail.com>
Signed-off-by: Pavol Loffay ploffay@redhat.com