-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[Language Text] Introduce the library #22640
[Language Text] Introduce the library #22640
Conversation
cca09c5
to
b6bf136
Compare
/azp run js - cognitivelanguage - ci |
No pipelines are associated with this pull request. |
/azp run prepare-pipelines |
Azure Pipelines successfully started running 1 pipeline(s). |
/azp run js - cognitivelanguage - ci |
Azure Pipelines successfully started running 1 pipeline(s). |
API change check APIView has identified API level changes in this PR and created following API reviews. |
@@ -0,0 +1,5 @@ | |||
# Release History | |||
|
|||
## 1.0.0-beta.1 (Unreleased) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think when creating a new package we try to have the new version be greater than the old version. Not sure if this was written into guidelines, but might be worth checking.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good point, I started an email thread about this and can address the issue in another PR.
/check-enforcer override |
Resets the code base for the Text Analytics library to the one used to release v5.1.0. The reset was done by checking out the library folder from the github tag for the v5.1.0 release. However, I updated the code base to use @azure/core-tracing@^1.0.0 too. This is needed because v6.0.0-beta.1 has been moved in #22640 into a new package: @azure/ai-language-text.
The archboard recommended creating a new library for the Language Text Analysis service and this PR copies the codebase of @azure/ai-text-analytics into the new @azure/ai-language-text. The only other change the PR makes is updating all the names/references accordingly. Please note that link verification will fail for many links that don't exist on the main branch yet and I plan to override it to get the PR merged.
TODO:
add configs for CI and resource creation