Skip to content

[Document] Py sdk 2026-05-15-preview#47984

Draft
amberchen-86 wants to merge 64 commits into
Azure:mainfrom
amberchen-86:v-amber/document-pysdk-2026-05-15-preview-continued
Draft

[Document] Py sdk 2026-05-15-preview#47984
amberchen-86 wants to merge 64 commits into
Azure:mainfrom
amberchen-86:v-amber/document-pysdk-2026-05-15-preview-continued

Conversation

@amberchen-86

Copy link
Copy Markdown
Member

Description

Please add an informative description that covers that changes made by the pull request and link all relevant issues.

If an SDK is being regenerated based on a new API spec, a link to the pull request containing these API spec changes should be included above.

All SDK Contribution checklist:

  • The pull request does not introduce [breaking changes]
  • CHANGELOG is updated for new features, bug fixes or other significant changes.
  • I have read the contribution guidelines.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

Copilot AI review requested due to automatic review settings July 9, 2026 21:30
@amberchen-86
amberchen-86 marked this pull request as ready for review July 16, 2026 17:07
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
9 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@quentinRobinson quentinRobinson left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@danieljurek danieljurek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the ci.yml changes for EngSys

@amberchen-86

Copy link
Copy Markdown
Member Author

/check-enforcer override

@msyyc msyyc left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This approval is just for ci.yml since I am one of owner about this file.

Copilot AI review requested due to automatic review settings July 17, 2026 18:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 57 out of 58 changed files in this pull request and generated 12 comments.

Comment on lines +165 to +166
final_response = pipeline_response.http_response
if final_response.status_code == 200:
Comment on lines +165 to +166
final_response = pipeline_response.http_response
if final_response.status_code == 200:
Comment on lines +11 to +12

__all__: list[str] = [] # Add all objects you want publicly available to users at this package level
Comment on lines +11 to +12

__all__: list[str] = [] # Add all objects you want publicly available to users at this package level
Comment on lines +119 to +123
def begin_submit_job( # type: ignore[override]
self,
body: Union[_models.AnalyzeDocumentsJob, JSON, IO[bytes]],
**kwargs: Any
) -> AnalyzeDocumentsLROPoller[ItemPaged[_models.Tasks]]:
Comment on lines +218 to +219
@distributed_trace_async
async def begin_cancel_job(self, job_id: str, **kwargs: Any) -> AsyncLROPoller[None]:
lro_delay = kwargs.pop("polling_interval", self._config.polling_interval)
cont_token: Optional[str] = kwargs.pop("continuation_token", None)
cls: ClsType[ItemPaged[_models.Tasks]] = kwargs.pop("cls", None)
kwargs.pop("error_map", None)
lro_delay = kwargs.pop("polling_interval", self._config.polling_interval)
cont_token = kwargs.pop("continuation_token", None)

kwargs.pop("error_map", None)
lro_delay = kwargs.pop("polling_interval", self._config.polling_interval)
cont_token: Optional[str] = kwargs.pop("continuation_token", None)
cls: ClsType[AsyncItemPaged[_models.Tasks]] = kwargs.pop("cls", None)
kwargs.pop("error_map", None)
lro_delay = kwargs.pop("polling_interval", self._config.polling_interval)
cont_token = kwargs.pop("continuation_token", None)

kwargs.pop("error_map", None)
@amberchen-86
amberchen-86 marked this pull request as draft July 17, 2026 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Cognitive - Language Community Contribution Community members are working on the issue customer-reported Issues that are reported by GitHub users external to the Azure organization.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants