-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fix changelog divergence for 2.x on main #7409
Conversation
be14fd0
to
815e037
Compare
815e037
to
87daf52
Compare
CHANGELOG.md
Outdated
@@ -86,13 +79,23 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), | |||
|
|||
## [Unreleased 2.x] | |||
### Added | |||
- [Extensions] Moving Extensions APIs to protobuf serialization. ([#6960](https://github.com/opensearch-project/OpenSearch/pull/6960)) | |||
- [Extensions] Add IdentityPlugin into core to support Extension identities ([#7246](https://github.com/opensearch-project/OpenSearch/pull/7246)) | |||
- [Extensions] Moving Extensions APIs to support cross versions via protobuf. ([#6960](https://github.com/opensearch-project/OpenSearch/pull/6960)) |
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.
My apologies, I've just merged in another PR #7403 which fixes this.
You might end up in merge conflicts.
1f5a962
to
b8829f8
Compare
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
CHANGELOG.md
Outdated
@@ -106,4 +109,4 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/), | |||
### Security | |||
|
|||
[Unreleased 3.0]: https://github.com/opensearch-project/OpenSearch/compare/2.x...HEAD | |||
[Unreleased 2.x]: https://github.com/opensearch-project/OpenSearch/compare/2.5...2.x | |||
[Unreleased 2.x]: https://github.com/opensearch-project/OpenSearch/compare/2....2.x |
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.
Lost 7
in 2.7
:/
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.
Sorry. Got drowned in too many branches. Updated
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## main #7409 +/- ##
=========================================
Coverage 70.53% 70.53%
Complexity 59554 59554
=========================================
Files 4877 4877
Lines 285807 285807
Branches 41164 41164
=========================================
Hits 201602 201602
Misses 67610 67610
Partials 16595 16595 |
Signed-off-by: Kunal Kotwani <[email protected]>
b8829f8
to
31b75fd
Compare
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
Signed-off-by: Kunal Kotwani <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
2.x
onmain
Related Issues
N/A
Check List
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.