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

Check S3 signer customization insensitively #2179

Merged

Conversation

syall
Copy link
Contributor

@syall syall commented Jul 7, 2023

For changes to files under the /codegen/aws-models folder, and manual edits to autogenerated code (e.g. /service/s3/api.go) please create an Issue instead of a PR for those type of changes.

N/A

If the PR addresses an existing bug or feature, please reference it here.

N/A

To help speed up the process and reduce the time to merge please ensure that Allow edits by maintainers is checked before submitting your PR. This will allow the project maintainers to make minor adjustments or improvements to the submitted PR, allow us to reduce the roundtrip time for merging your request.

N/A


Check signer customizations insensitively, specifically for S3 and EventBridge.

@syall syall requested a review from a team as a code owner July 7, 2023 22:09
@syall syall merged commit 168064f into aws:feat-ep20 Jul 7, 2023
isaiahvita pushed a commit to isaiahvita/aws-sdk-go-v2 that referenced this pull request Jul 25, 2023
* Check S3 signer customization insensitively
* Check EventBridge signer customization insensitively
isaiahvita pushed a commit that referenced this pull request Jul 25, 2023
* Check S3 signer customization insensitively
* Check EventBridge signer customization insensitively
isaiahvita pushed a commit that referenced this pull request Jul 27, 2023
* Check S3 signer customization insensitively
* Check EventBridge signer customization insensitively
isaiahvita pushed a commit that referenced this pull request Jul 31, 2023
* Check S3 signer customization insensitively
* Check EventBridge signer customization insensitively
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.

3 participants