-
Notifications
You must be signed in to change notification settings - Fork 24
Release: smithy_http-0.2.0, smithy_aws_core-0.1.1 #580
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
Changes from all commits
06b21df
f3762ce
4486123
7bd1767
c41c78b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| { | ||
| "changes": [ | ||
| { | ||
| "type": "dependency", | ||
| "description": "Bump `smithy-http` from `~=0.1.0` to `~=0.2.0`." | ||
| } | ||
| ] | ||
| } |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,4 @@ | ||
| # Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
|
|
||
| __version__ = "0.1.0" | ||
| __version__ = "0.1.1" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,8 @@ | ||
| { | ||
| "changes": [ | ||
| { | ||
| "type": "breaking", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Also for my understanding, did we canonicalize "breaking" as an official type going forward? Is this for pre-1.0 or is the intent to use this after that?
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Yup, the allow list is here: https://github.com/smithy-lang/smithy-python/blob/develop/scripts/changelog/new-entry.py#L58. Boto3 uses
|
||
| "description": "Update `AWSCRTHTTPClient` to integrate with the new AWS CRT async interfaces. ([#573](https://github.com/smithy-lang/smithy-python/pull/573)). The `AWSCRTHTTPResponse` constructor now accepts a `stream` argument of type `awscrt.aio.http.AIOHttpClientStreamUnified`. Additionally, the following classes were removed: `CRTResponseBody`, `CRTResponseFactory`, and `BufferableByteStream`." | ||
| } | ||
| ] | ||
| } | ||
This file was deleted.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need to be careful trying to strictly pin this. We did it in boto3 because it was an optional opt in. Now that the CRT may be a required dependency we should be trying to broaden the range while still making sure it works.
Is this the earliest version that supports the new APIs?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yea, we depend on the changes required in awscrt v0.28.2. I mentioned that here which is where we actually define this dependency. This change here is actually not used anywhere. I'm just updating here since we may use this code in the future (if we decide to make CRT a required dependency for all clients). We can also consider removing it until we need it again.