Skip to content
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

[Backport 2.x] Correct Java Native Access Description in THIRD-PARTY.txt #2111

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 2a00e2b from #2107

Signed-off-by: Stephen Crawford <steecraw@amazon.com>
(cherry picked from commit 2a00e2b)
@opensearch-trigger-bot opensearch-trigger-bot bot requested a review from a team September 27, 2022 15:48
@codecov-commenter
Copy link

codecov-commenter commented Sep 27, 2022

Codecov Report

Merging #2111 (76c68f6) into 2.x (adbeb36) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                2.x    #2111   +/-   ##
=========================================
  Coverage     61.03%   61.03%           
  Complexity     3228     3228           
=========================================
  Files           256      256           
  Lines         18080    18080           
  Branches       3225     3225           
=========================================
  Hits          11036    11036           
  Misses         5467     5467           
  Partials       1577     1577           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Member

@peternied peternied left a comment

Choose a reason for hiding this comment

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

This boils down to a documentation change which is only really valid from our main branch, I don't think we should backport this change. I think we should close this pull request without merging, how does this occur to you @cwperks?

@cwperks
Copy link
Member

cwperks commented Sep 27, 2022

@peternied As a rule of thumb, I have been backporting any change to unreleased 2.x unless it is a breaking change that cannot be released on the 2.x line.

@peternied
Copy link
Member

Spoke offline with @cwperks, closing this out since it isn't need in backport versions

@peternied peternied closed this Sep 27, 2022
@peternied peternied deleted the backport/backport-2107-to-2.x branch November 30, 2022 21:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants