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] adding Mingshi Liu as a maintainer so she can be release manager (#112) #126

Closed
wants to merge 1 commit into from

Conversation

mingshl
Copy link
Collaborator

@mingshl mingshl commented Jan 9, 2023

Backport 'efaaeff' from 112

@mingshl mingshl requested a review from a team January 9, 2023 22:50
@mingshl mingshl self-assigned this Jan 9, 2023
@mingshl mingshl changed the title adding Mingshi Liu as a maintainer so she can be release manager (#112) [Backport 2.x] adding Mingshi Liu as a maintainer so she can be release manager (#112) Jan 9, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #126 (982f3ec) into 2.x (09236bf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              2.x     #126   +/-   ##
=======================================
  Coverage   81.42%   81.42%           
=======================================
  Files          13       13           
  Lines         140      140           
  Branches       27       27           
=======================================
  Hits          114      114           
  Misses         24       24           
  Partials        2        2           
Flag Coverage Δ
dashboards-search-relevance 81.42% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@noCharger
Copy link
Collaborator

noCharger commented Jan 9, 2023

we should remove the backport label for a backport PR https://github.com/opensearch-project/OpenSearch-Dashboards/blob/main/RELEASING.md#how-to-validate-that-merged-commits-have-been-properly-backported

Also we should avoid backport manually. Add the label to the original pr on main branch and it will create the backport PR automatically

@noCharger
Copy link
Collaborator

Please close this one and use #127

@mingshl
Copy link
Collaborator Author

mingshl commented Jan 9, 2023

closed the manual backport.

@mingshl mingshl closed this Jan 9, 2023
@mingshl mingshl deleted the backport-112-2.x branch January 9, 2023 23:08
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.

5 participants