-
Notifications
You must be signed in to change notification settings - Fork 123
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
Add missing types for AwsSigv4SignerOptions.service #374
Conversation
2.2.0 added support for AOSS, but the types weren't updated.
Hi @magoz can you sign off your commit with |
Will need a line in https://github.com/opensearch-project/opensearch-js/blob/main/CHANGELOG.md, too. |
This reverts commit 0bb24a4.
Signed-off-by: magoz <[email protected]>
I cannot seem to be able to sign off the initial commit. I followed the instructions, but when I I originally added the PR via the web interface, and I suspect it sees me as a different user when I push from my local machine. I tried to revert, but it didn't help since all the commits in the PR must be signed. Any advice on how to fix this? Should I create a new PR with the signed commit? |
@magoz Create a new branch, add the changes, commit with DCO, then create a new PR. See if that would work for ya. |
@dblock TypeScript would catch this. This repo needs to be converted to TS and we can do that along with API code generation for this client. |
Description
2.2.0 added support for AOSS, but the types weren't updated.
Issues Resolved
Fixes #356
Fixes #366
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.