Skip to content

Commit

Permalink
fix(python): add request options parameter name to readme snippets (#…
Browse files Browse the repository at this point in the history
  • Loading branch information
armandobelardo committed Sep 26, 2024
1 parent 90643d4 commit fe56df4
Show file tree
Hide file tree
Showing 85 changed files with 173 additions and 271 deletions.
8 changes: 7 additions & 1 deletion generators/python/sdk/versions.yml
Original file line number Diff line number Diff line change
@@ -1,4 +1,10 @@
# For unreleased changes, use unreleased.yml
# For unreleased changes, use unreleased.yml
- version: 4.2.7-rc4
irVersion: 53
changelogEntry:
- type: fix
summary: Generated readmes now include the parameter name required for the request options parameter within the example snippets.

- version: 4.2.7-rc3
irVersion: 53
changelogEntry:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -136,7 +136,7 @@ def _client_writer(writer: AST.NodeWriter) -> None:
{client_instantiation_str}
# Override timeout for a specific method
client.{endpoint.endpoint_package_path}{endpoint.method_name}({"..., " if has_parameters else ""}{{
client.{endpoint.endpoint_package_path}{endpoint.method_name}({"..., " if has_parameters else ""}request_options={{
"timeout_in_seconds": 1
}})
"""
Expand Down Expand Up @@ -190,7 +190,7 @@ def _build_retries_snippets(self) -> List[str]:
if endpoint is not None:
has_parameters = self._endpoint_metadata.has_parameters(endpoint_id)
retry_snippets.append(
f"""client.{endpoint.endpoint_package_path}{endpoint.method_name}({"..., " if has_parameters else ""}{{
f"""client.{endpoint.endpoint_package_path}{endpoint.method_name}({"..., " if has_parameters else ""}request_options={{
"max_retries": 1
}})
"""
Expand Down
4 changes: 2 additions & 2 deletions seed/python-sdk/alias-extends/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/alias/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/any-auth/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/api-wide-base-path/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/audiences/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/auth-environment-variables/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/basic-auth-environment-variables/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/basic-auth/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/bearer-token-environment-variable/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/bytes/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/cross-package-type-names/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/custom-auth/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/enum/no-custom-config/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/enum/real-enum-forward-compat/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/enum/real-enum/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/enum/strenum/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/error-property/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/examples/client-filename/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/examples/no-custom-config/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions seed/python-sdk/examples/readme/README.md

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit fe56df4

Please sign in to comment.