Dev cognitiveservices text analytics v3.1 preview.4#12978
Merged
Conversation
…w.3 to version v3.1-preview.4
* Adding changed example * json change for om part 1 * Change API structure as well * refine text again * Getting back to false by default * word chagne * . * final wording change * fix for readme.md Co-authored-by: josuh <josuh@microsoft.com>
* Made results array optional since tasks can be in a failed state with no results * Fixing a syntax issue in the README * Attempting to address an Autorest error in the README * Attempting to address an Autorest error in the README * Added some info about v3.1-preview.4 to the python readme file; attempting to correct syntax issues in readme.md * Attempting to fix generation for the Python SDK * Attempting to fix generation for the Python SDK Co-authored-by: Abigail Hartman <Abigail.Hartman@microsoft.com>
#12340) * updated examples and health definitions for new preview version * fixed linting issue in readme * removed old properties in ta.json * undo changes to sentiment portion of response, update health response * fixed spelling issues * fixed more typose * remove linting issue in readme * added enums to assertions * add in ms-enum extension * ran prettier json linter over files * made assertion field singular
* based off of correct branch now * change the request from enum to a generic list of strings * cleanup some other comments * add enum type * change type --> category * add categories to the swagger * update the types to make them read better * add in the NER types * add in address type as well * fix some typos * one last typo
* updated examples and health definitions for new preview version * fixed linting issue in readme * removed old properties in ta.json * undo changes to sentiment portion of response, update health response * fixed spelling issues * fixed more typose * remove linting issue in readme * added enums to assertions * add in ms-enum extension * ran prettier json linter over files * made assertion field singular * addressed comments and updated description
…338 (#12650) * updated examples and health definitions for new preview version * fixed linting issue in readme * removed old properties in ta.json * undo changes to sentiment portion of response, update health response * fixed spelling issues * fixed more typose * remove linting issue in readme * added enums to assertions * add in ms-enum extension * ran prettier json linter over files * made assertion field singular * addressed comments and updated description * addressed comments for minor syntax changes * replaced all instances of well-known * fixed linting issue - pii category was set to list instead of string
…12660) * Trying to solve swagger validation issues * Removed x-ms-enum property from PiiCategories * Created a definition for PiiCategories in addition to a parameter in order to support passing the same thing in the body of a Task object * Ran 'prettier' to, uh... make it prettier. * Updated custom-words.txt * Corrected typos in Swagger and removed unneeded entries in custom-words.txt * Changed 'default' to 'x-ms-client-default' in order to pass both model validation and linting * Fixed a misspelled PII category * Reran prettier check * Removed an experimental line in the Swagger * Adding collectionFormat to fix an autorest error * Removing x-ms-client-default Co-authored-by: Abigail Hartman <Abigail.Hartman@microsoft.com>
…es text analytics v3.1 preview.4 (#12737) * updated examples and health definitions for new preview version * fixed linting issue in readme * removed old properties in ta.json * undo changes to sentiment portion of response, update health response * fixed spelling issues * fixed more typose * remove linting issue in readme * added enums to assertions * add in ms-enum extension * ran prettier json linter over files * made assertion field singular * addressed comments and updated description * addressed comments for minor syntax changes * replaced all instances of well-known * fixed linting issue - pii category was set to list instead of string * revert change to pii default * remove default value as this is an optional parameter * added extensible list to relation types for healthcare * revert change to pii default * changed enum type and made extensible via the modelAsString param
|
Hi, @anuchandy Thanks for your PR. I am workflow bot for review process. Here are some small tips. Any feedback about review process or workflow bot, pls contact swagger and tools team. vsswagger@microsoft.com |
Swagger Validation Report
|
| Rule | Message |
|---|---|
| Guid used in model definition 'JobMetadata' for property 'jobId'. Usage of Guid is not recommanded. If GUIDs are absolutely required in your service, please get sign off from the Azure API review board. Location: TextAnalytics/preview/v3.1-preview.4/TextAnalytics.json#L2037 |
️⌛Avocado pending [Detail]
️⌛ModelValidation pending [Detail]
️⌛SemanticValidation pending [Detail]
️⌛[Staging] Cross Version BreakingChange (Base on preview version) pending [Detail]
️⌛[Staging] Cross Version BreakingChange (Base on stable version) pending [Detail]
️⌛CredScan pending [Detail]
️⌛[Staging] SDK Track2 Validation pending [Detail]
️⌛[Staging] PrettierCheck pending [Detail]
️⌛[Staging] SpellCheck pending [Detail]
Swagger Generation Artifacts
|
tjprescott
approved these changes
Feb 12, 2021
Member
Author
|
/azp run |
|
Azure Pipelines successfully started running 2 pipeline(s). |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
Changelog
Please ensure to add changelog with this PR by answering the following questions.
Contribution checklist:
If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist
Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.
Please ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.
If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from Breaking Change Review Board as defined in the Breaking Change Policy.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Addition details on the process and office hours are on the Breaking change Wiki.
Please follow the link to find more details on PR review process.