-
Notifications
You must be signed in to change notification settings - Fork 16
Add generated Lex Runtime V2 client with integration tests #51
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
base: develop
Are you sure you want to change the base?
Changes from all commits
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,4 @@ | ||
| { | ||
| "type": "api-change", | ||
| "description": "Initial client release with support for current Amazon Lex Runtime V2 operations." | ||
| } | ||
|
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. Same comment as before. This should be done by our release automation. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,6 @@ | ||
| # Changelog | ||
|
|
||
| ## v0.4.0 | ||
|
|
||
| ### Features | ||
| * Initial client release with support for current Amazon Lex Runtime V2 operations. |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| # Code generated by smithy-python-codegen DO NOT EDIT. | ||
|
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. Anyway we can avoid this header on the package README? This will also appear on the PyPI page for the package. |
||
|
|
||
| ## Amazon Lex Runtime V2 Client | ||
|
|
||
| aws_sdk_lex_runtime_v2 client | ||
|
|
||
| ### Documentation | ||
|
|
||
| This section contains documentation for the Amazon Lex V2 Runtime V2 API | ||
| operations. | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,61 @@ | ||
| # Code generated by smithy-python-codegen DO NOT EDIT. | ||
|
|
||
| [project] | ||
| name = "aws_sdk_lex_runtime_v2" | ||
| version = "0.4.0" | ||
| description = "aws_sdk_lex_runtime_v2 client" | ||
| readme = "README.md" | ||
| requires-python = ">=3.12" | ||
| keywords = ["smithy", "aws_sdk_lex_runtime_v2"] | ||
| license = {text = "Apache-2.0"} | ||
| classifiers = [ | ||
| "Development Status :: 2 - Pre-Alpha", | ||
| "Intended Audience :: Developers", | ||
| "Intended Audience :: System Administrators", | ||
| "Natural Language :: English", | ||
| "License :: OSI Approved :: Apache Software License", | ||
| "Programming Language :: Python", | ||
| "Programming Language :: Python :: 3", | ||
| "Programming Language :: Python :: 3 :: Only", | ||
| "Programming Language :: Python :: 3.12", | ||
| "Programming Language :: Python :: 3.13", | ||
| "Programming Language :: Python :: 3.14" | ||
| ] | ||
|
|
||
| dependencies = [ | ||
| "smithy_aws_core[eventstream, json]~=0.4.0", | ||
| "smithy_core~=0.3.0", | ||
|
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. When I pulled down these changes, installed the client locally, and tried to run the integ tests, I got this error: This is because the I think this client was created in between the actual release of smithy-core and the codegen merge. Let's regenerate the client using the latest release to make sure all dependencies are up-to-date for testing. |
||
| "smithy_http[awscrt]~=0.3.0" | ||
| ] | ||
|
|
||
| [dependency-groups] | ||
| test = [ | ||
| "pytest>=9.0.1,<10.0.0", | ||
| "pytest-asyncio>=1.3.0,<1.4.0" | ||
| ] | ||
|
|
||
| [build-system] | ||
| requires = ["hatchling"] | ||
| build-backend = "hatchling.build" | ||
|
|
||
| [tool.pyright] | ||
| typeCheckingMode = "strict" | ||
| reportPrivateUsage = false | ||
| reportUnusedFunction = false | ||
| reportUnusedVariable = false | ||
| reportUnnecessaryComparison = false | ||
| reportUnusedClass = false | ||
| enableExperimentalFeatures = true | ||
|
|
||
| [tool.ruff] | ||
| target-version = "py312" | ||
|
|
||
| [tool.ruff.lint] | ||
| ignore = ["F841"] | ||
|
|
||
| [tool.ruff.format] | ||
| skip-magic-trailing-comma = true | ||
|
|
||
| [tool.pytest.ini_options] | ||
| python_classes = ["!Test"] | ||
| asyncio_mode = "auto" | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,3 @@ | ||
| # Code generated by smithy-python-codegen DO NOT EDIT. | ||
|
|
||
| __version__: str = "0.4.0" |
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1 @@ | ||
| # Code generated by smithy-python-codegen DO NOT EDIT. |
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.
Do we need this changelog entry? Our release automation should handle new client setup, including initial versioning and changelog entries. We will have merge conflicts if the wording if different or we may have repeated entries because the uuid in the newly generated entry is different.