Skip to content

Conversation

@gaobinlong
Copy link
Contributor

Description

There're two items in the changelog have same PR number, the first one is not correct.

Related Issues

No issue.

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

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.

Signed-off-by: Binlong Gao <gbinlong@amazon.com>
@gaobinlong gaobinlong requested a review from a team as a code owner October 23, 2025 02:36
Copy link
Member

@sandeshkr419 sandeshkr419 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, I missed noticing this before merging the change.

Copy link
Contributor

@jainankitk jainankitk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sandeshkr419 - One of the commit is missing the signoff causing DCO check to fail.

@sandeshkr419
Copy link
Member

Weird, probably the first time where my commit from UI failed DCO.

@sandeshkr419 sandeshkr419 merged commit 29b113a into opensearch-project:main Oct 23, 2025
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants