Skip to content

fix(amazonq): add handling for new limit mechanism #6091

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Nov 22, 2024
Merged

fix(amazonq): add handling for new limit mechanism #6091

merged 1 commit into from
Nov 22, 2024

Conversation

@willyyhuang
Copy link
Contributor

@willyyhuang willyyhuang commented Nov 22, 2024

Problem

RTS changed feature dev governance interface on dealing with quota

Solution

Added handling for this case on feature dev

License: I confirm that my contribution is made under the terms of the Apache 2.0 license.

@willyyhuang willyyhuang requested a review from a team as a code owner November 22, 2024 17:20
@github-actions
Copy link

github-actions bot commented Nov 22, 2024

  • This pull request modifies code in src/* but no tests were added/updated.
    • Confirm whether tests should be added or ensure the PR description explains why tests are not required.

@willyyhuang willyyhuang force-pushed the wlsun/add-new-limit-error-handling-case branch from 71be951 to 6bdaef6 Compare November 22, 2024 17:29
@jpinkney-aws jpinkney-aws changed the title fix(featureDev): add handling for new limit mechanism fix(amazonq): add handling for new limit mechanism Nov 22, 2024
@jpinkney-aws jpinkney-aws merged commit 7282c40 into aws:master Nov 22, 2024
28 of 31 checks passed
@willyyhuang willyyhuang mentioned this pull request Nov 22, 2024
Merged
jpinkney-aws pushed a commit that referenced this pull request Nov 22, 2024
@willyyhuang
fix(amazonq): add throttling exception handling and in code comments …
a0de73d 
…for featureDev quota (#6093)

## Problem
- Follow up of #6091 
- improve and refactor the code from that previous PR

## Solution
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers

@jpinkney-aws jpinkney-aws jpinkney-aws approved these changes

@chengoramazon chengoramazon chengoramazon approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@willyyhuang @jpinkney-aws @chengoramazon