-
Notifications
You must be signed in to change notification settings - Fork 54
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
Relase Notes 2.11.1 #1262
Relase Notes 2.11.1 #1262
Conversation
Signed-off-by: Peter Fitzgibbons <peter.fitzgibbons@gmail.com>
|
||
### Features | ||
|
||
- [Feature] Match discover look and feel by @opensearch-trigger-bot in ([#1179](https://github.com/opensearch-project/dashboards-observability/pull/1179) |
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.
Nit: would be nice to be able to link the original author instead of the backport bot
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.11 #1262 +/- ##
===========================================
+ Coverage 44.22% 56.81% +12.58%
===========================================
Files 327 325 -2
Lines 19477 11739 -7738
Branches 4643 2699 -1944
===========================================
- Hits 8614 6669 -1945
+ Misses 10818 5025 -5793
Partials 45 45
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
||
### Features | ||
|
||
- [Feature] Match discover look and feel by @opensearch-trigger-bot in ([#1179](https://github.com/opensearch-project/dashboards-observability/pull/1179) |
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.
Feature? In a PATCH release?
https://semver.org/#spec-item-6
Patch version Z (x.y.Z | x > 0) MUST be incremented if only backward compatible bug fixes are introduced. A bug fix is defined as an internal change that fixes incorrect behavior.
Have not all non-bugfix backports been revert?
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.
Looks like they have an the release notes were fixed in #1273. This was an oversight. There's also some confusion on why this is under a "refactor" section, but either way it got reverted from the patch release. Sorry about that.
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.
should not be a feature / can you double check please
#1179 was part of the revert commit. Signed-off-by: Peter Fitzgibbons <peter.fitzgibbons@gmail.com>
@smortex Thanks. These notes are looking incorrect. |
…roject#1266) Signed-off-by: Craig Perkins <cwperx@amazon.com> Co-authored-by: Peter Nied <petern@amazon.com> (cherry picked from commit 4cb6252090696b81fcd9410b74a830479fabbebb)
Description
Release Notes 2.11.1
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.