-
Notifications
You must be signed in to change notification settings - Fork 146
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] Manual backport for adding AWS Rekognition trust endpoint #3445
[Backport 2.x] Manual backport for adding AWS Rekognition trust endpoint #3445
Conversation
* feat: add rekognition trust endpoint Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com> * refactor: consistent regex with previous connectors and looser checks on region Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com> * test: add test case to validate connector creation successful Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com> * chore: spotless apply Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com> * chore: linter fixes Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com> --------- Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>
Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>
Not sure why spotless is failing, I have applied it, checking |
@pyek-bot , could you resolve the conflicts? |
Signed-off-by: Pavan Yekbote <mail2pavanyekbote@gmail.com>
Conflict due to new merges, fixed it again |
|
I saw some other buildings with the same error here. It seems not related to this pr. |
Found this github issue related to this: diffplug/spotless#1783 |
This is also affecting my builds :( |
Description
Fixes conflicts and backports PR that adds Amazon Rekognition as a trust endpoint
Main PR: #3419
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.