-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
BREAKING(aws): change the RESTMethodSettings
type to a list of settings
#6432
Comments
is this about changing the input structure of the check? if so 1) need to update the schema as well 2) isn't it a breaking change? |
@itaysk
|
yes, the title should be updated, and we should remember to mention it in the release notes. also, I think we should be more minded that changing input structure is a breaking change and follow a procedure:
I understand this might be a low impact change so we can fast-forward through some of the process, but we should at least be intentional about it |
@itaysk @nikpivkin we'll err on the side of caution and mark this as a breaking change. We don't have usage metrics so can't articulate how many users it will affect or how widely it is used (it's a non required argument). |
RESTMethodSettings
type to a list of settingsRESTMethodSettings
type to a list of settings
Several settings can be specified for the API stage.
The text was updated successfully, but these errors were encountered: