Skip to content
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

Sort imports in service template #479

Merged
merged 1 commit into from
Dec 14, 2023
Merged

Sort imports in service template #479

merged 1 commit into from
Dec 14, 2023

Conversation

hectorcast-db
Copy link
Contributor

Changes

Sort imports in service template. The following line must be the first command in a Python file:
from __future__ import annotations
Usually this is sorted by the linter. But if the file generated is not syntactically correct, it won't be sorted and it will cause a error which will mask the real error with the generated file.

Tests

deco openapi generate-sdk py --openapi-spec master

  • make test run locally
  • make fmt applied
  • relevant integration tests applied

@hectorcast-db hectorcast-db added this pull request to the merge queue Dec 14, 2023
Merged via the queue into main with commit 1ad9a12 Dec 14, 2023
9 checks passed
@hectorcast-db hectorcast-db deleted the sort-import branch December 14, 2023 10:59
kartikgupta-db added a commit that referenced this pull request Dec 20, 2023
* Sort imports in service template ([#479](#479)).
* Add `py.typed` to support PEP-561 ([#483](#483)).
* Fixed bug in `@retried` when exception subtypes were not respected ([#484](#484)).
* Make `WorkspaceClient` and `AccountClient` more friendly with autospeccing ([#480](#480)).

API Changes:

 * Added `azure_workspace_info` field for `databricks.sdk.service.provisioning.Workspace`.
 * Added `databricks.sdk.service.provisioning.AzureWorkspaceInfo` dataclass.
 * Changed `update_config()` method for [w.serving_endpoints](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/serving_endpoints.html) workspace-level service with new required argument order.
 * Changed `served_entities` field for `databricks.sdk.service.serving.EndpointCoreConfigInput` to no longer be required.
 * Changed `create()` method for [a.account_ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/account/account_ip_access_lists.html) account-level service with new required argument order.
 * Changed `replace()` method for [a.account_ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/account/account_ip_access_lists.html) account-level service with new required argument order.
 * Changed `update()` method for [a.account_ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/account/account_ip_access_lists.html) account-level service with new required argument order.
 * Changed `create()` method for [w.ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/ip_access_lists.html) workspace-level service with new required argument order.
 * Changed `replace()` method for [w.ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/ip_access_lists.html) workspace-level service with new required argument order.
 * Changed `update()` method for [w.ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/ip_access_lists.html) workspace-level service with new required argument order.
 * Changed `ip_addresses` field for `databricks.sdk.service.settings.CreateIpAccessList` to no longer be required.
 * Changed `ip_addresses` field for `databricks.sdk.service.settings.ReplaceIpAccessList` to no longer be required.
 * Removed `list_id` field for `databricks.sdk.service.settings.ReplaceIpAccessList`.
 * Changed `enabled` field for `databricks.sdk.service.settings.UpdateIpAccessList` to no longer be required.
 * Changed `ip_addresses` field for `databricks.sdk.service.settings.UpdateIpAccessList` to no longer be required.
 * Changed `label` field for `databricks.sdk.service.settings.UpdateIpAccessList` to no longer be required.
 * Removed `list_id` field for `databricks.sdk.service.settings.UpdateIpAccessList`.
 * Changed `list_type` field for `databricks.sdk.service.settings.UpdateIpAccessList` to no longer be required.

OpenAPI SHA: d3853c8dee5806d04da2ae8910f273ffb35719a5, Date: 2023-12-14
@kartikgupta-db kartikgupta-db mentioned this pull request Dec 20, 2023
github-merge-queue bot pushed a commit that referenced this pull request Dec 20, 2023
* Sort imports in service template
([#479](#479)).
* Add `py.typed` to support PEP-561
([#483](#483)).
* Fixed bug in `@retried` when exception subtypes were not respected
([#484](#484)).
* Make `WorkspaceClient` and `AccountClient` more friendly with
autospeccing
([#480](#480)).

API Changes:

* Added `azure_workspace_info` field for
`databricks.sdk.service.provisioning.Workspace`.
* Added `databricks.sdk.service.provisioning.AzureWorkspaceInfo`
dataclass.
* Changed `update_config()` method for
[w.serving_endpoints](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/serving_endpoints.html)
workspace-level service with new required argument order.
* Changed `served_entities` field for
`databricks.sdk.service.serving.EndpointCoreConfigInput` to no longer be
required.
* Changed `create()` method for
[a.account_ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/account/account_ip_access_lists.html)
account-level service with new required argument order.
* Changed `replace()` method for
[a.account_ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/account/account_ip_access_lists.html)
account-level service with new required argument order.
* Changed `update()` method for
[a.account_ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/account/account_ip_access_lists.html)
account-level service with new required argument order.
* Changed `create()` method for
[w.ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/ip_access_lists.html)
workspace-level service with new required argument order.
* Changed `replace()` method for
[w.ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/ip_access_lists.html)
workspace-level service with new required argument order.
* Changed `update()` method for
[w.ip_access_lists](https://databricks-sdk-py.readthedocs.io/en/latest/workspace/ip_access_lists.html)
workspace-level service with new required argument order.
* Changed `ip_addresses` field for
`databricks.sdk.service.settings.CreateIpAccessList` to no longer be
required.
* Changed `ip_addresses` field for
`databricks.sdk.service.settings.ReplaceIpAccessList` to no longer be
required.
* Removed `list_id` field for
`databricks.sdk.service.settings.ReplaceIpAccessList`.
* Changed `enabled` field for
`databricks.sdk.service.settings.UpdateIpAccessList` to no longer be
required.
* Changed `ip_addresses` field for
`databricks.sdk.service.settings.UpdateIpAccessList` to no longer be
required.
* Changed `label` field for
`databricks.sdk.service.settings.UpdateIpAccessList` to no longer be
required.
* Removed `list_id` field for
`databricks.sdk.service.settings.UpdateIpAccessList`.
* Changed `list_type` field for
`databricks.sdk.service.settings.UpdateIpAccessList` to no longer be
required.

OpenAPI SHA: d3853c8dee5806d04da2ae8910f273ffb35719a5, Date: 2023-12-14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants