-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
Add generated code and SDK API tests for Personalizer v1.0 preview #5944
Conversation
src/SDKs/CognitiveServices/dataPlane/Personalizer/Personalizer/Personalizer.csproj
Outdated
Show resolved
Hide resolved
src/SDKs/CognitiveServices/dataPlane/Personalizer/Personalizer/Personalizer.csproj
Outdated
Show resolved
Hide resolved
src/SDKs/CognitiveServices/dataPlane/Personalizer/Personalizer/Properties/AssemblyInfo.cs
Outdated
Show resolved
Hide resolved
src/SDKs/CognitiveServices/dataPlane/Personalizer/Personalizer/Properties/AssemblyInfo.cs
Outdated
Show resolved
Hide resolved
src/SDKs/CognitiveServices/dataPlane/Personalizer/Personalizer/generate.ps1
Outdated
Show resolved
Hide resolved
src/SDKs/CognitiveServices/dataPlane/Personalizer/Personalizer/generate.ps1
Outdated
Show resolved
Hide resolved
Created for Azure/azure-rest-api-specs#5743 |
src/SDKs/CognitiveServices/dataPlane/Personalizer/Personalizer/Personalizer.csproj
Outdated
Show resolved
Hide resolved
@@ -3,7 +3,7 @@ | |||
<PropertyGroup> | |||
<Description>Microsoft.Azure.CognitiveServices.Personalizer.Tests Class Library</Description> | |||
<AssemblyName>Microsoft.Azure.CognitiveServices.Personalizer.Tests</AssemblyName> | |||
<Version>1.0.0</Version> | |||
<Version>0.8.0-preview</Version> |
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.
Test project version must be set to 1.0.0 and never modified. Not blocking but must be fixed
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.
fixed
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.
LGTM will merge once CIs pass
…zure#5944) * Initial setup of Personalizer SDK * checking in Rank Test * Add all API tests except evalations * Consolidate tests to controller groups. * Add evaluations tests. * Adding Readme and generate.ps1 * regenerate client and fix tests * fix all tests , add sdkinfo and metadata files * fix version names * add tests proj
Add generated code and SDK API tests for Personalizer v1.0 preview
Spec PR links
Azure/azure-rest-api-specs#5743
Azure/azure-rest-api-specs#5777
Both Spec PRs have been merged