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

[Text Translation] Updating the SDK from the latest TypeSpec #35450

Merged
merged 33 commits into from
May 21, 2024

Conversation

MikeyMCZ
Copy link
Member

@MikeyMCZ MikeyMCZ commented May 1, 2024

Description

Updating the SDK from the latest TypeSpec.

  • Updating names of the models based on the GA review feedback
  • Updating the names of custom paths to comply with Py naming

@azure-sdk
Copy link
Collaborator

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-ai-translation-text

Copy link
Member

@kristapratico kristapratico left a comment

Choose a reason for hiding this comment

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

@msyyc
Copy link
Member

msyyc commented May 20, 2024

Overall looks good except CI fails: https://github.com/Azure/azure-sdk-for-python/pull/35450/checks?check_run_id=25122625367. There is detailed log and please fix it with those error info.

Copy link
Member

@kristapratico kristapratico left a comment

Choose a reason for hiding this comment

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

Leaving comments on how to fix CI errors

Copy link
Member

@kristapratico kristapratico left a comment

Choose a reason for hiding this comment

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

Looking good! Few small things

MikeyMCZ and others added 2 commits May 21, 2024 10:46
…/text/_patch.py

Co-authored-by: Krista Pratico <krpratic@microsoft.com>
@kristapratico kristapratico merged commit 802609d into Azure:main May 21, 2024
17 checks passed
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.

4 participants