-
Notifications
You must be signed in to change notification settings - Fork 58
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
Updated DEV guide with the new initialization api #820
Updated DEV guide with the new initialization api #820
Conversation
Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>
@kolchfa-aws not able to add you as a reviewer, so tagging here for a review. Thanks! |
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.
LGTM with a couple of comments
242612b
to
d618a71
Compare
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.
Besides @ryanbogan comments, lgtm
Signed-off-by: Owais Kazi <owaiskazi19@gmail.com>
d618a71
to
17c327d
Compare
LGTM! Waiting to merge until the companion PR is in |
Companion PR is in! |
* Updated DEV guide with the new initialization api Signed-off-by: Owais Kazi <owaiskazi19@gmail.com> * Addressed PR comments Signed-off-by: Owais Kazi <owaiskazi19@gmail.com> --------- Signed-off-by: Owais Kazi <owaiskazi19@gmail.com> (cherry picked from commit bef933f) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
* Updated DEV guide with the new initialization api * Addressed PR comments --------- (cherry picked from commit bef933f) Signed-off-by: Owais Kazi <owaiskazi19@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Companion PR for: opensearch-project/OpenSearch#8029
Issues Resolved
Part of #782
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.