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 Analytics] Merge feature branch for v3.2-preview.1 #16716

Merged
merged 7 commits into from
Aug 4, 2021

Conversation

deyaaeldeen
Copy link
Member

@deyaaeldeen deyaaeldeen commented Aug 3, 2021

This PR merges the feature branch for adding the support for extract summary actions. This branch has one unreviewed commit which has re-recordings of all tests: 143c35d

Fixes #16415.

* [Text Analytics] Add support for extract summary actions

* add a test case

* update readme

* edit

* edit

* edit

* regenerate using latest swagger

* edit

* edit

* address feedback

* edit

* edit
@deyaaeldeen
Copy link
Member Author

/azp run js - textanalytics - ci

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@deyaaeldeen deyaaeldeen force-pushed the feature/textanalytics/3.2 branch from 7a6b2e0 to 93de9ad Compare August 3, 2021 20:47
@@ -252,7 +252,7 @@ export interface ExtractKeyPhrasesSuccessResult extends TextAnalyticsSuccessResu
export interface ExtractSummaryAction extends TextAnalyticsAction {
disableServiceLogs?: boolean;
maxSentenceCount?: number;
orderBy: string;
orderBy?: string;
Copy link
Member

Choose a reason for hiding this comment

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

good catch!

@deyaaeldeen deyaaeldeen enabled auto-merge (squash) August 3, 2021 23:13
@deyaaeldeen deyaaeldeen merged commit 0d44620 into main Aug 4, 2021
@deyaaeldeen deyaaeldeen deleted the feature/textanalytics/3.2 branch August 4, 2021 00:09
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-js that referenced this pull request Nov 18, 2021
add 2021-05-01-preview/ServerConnectionPolicies.json to V5 SDK tag (Azure#16716)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Text Analytics] Add support for extractive summarization tasks
2 participants