Skip to content

Conversation

@stevengum
Copy link

Request to release new BotService SDK via #12704

@stevengum stevengum requested a review from erich-wang as a code owner July 16, 2020 20:49
Copy link
Member

@isra-fel isra-fel left a comment

Choose a reason for hiding this comment

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

Hi @stevengum , plsase

  • Check in the metadata TXT file generated when you run generate.ps1(details)
  • Update api tag in AzSdk.RP.props, release notes in csproj
  • Update versions in csproj and assemblyinfo.cs if you are going to release

Thanks

@openapi-sdkautomation openapi-sdkautomation bot force-pushed the sdkAutomation/Microsoft.Azure.Management.BotService branch from 7c4a4ae to b02a6c1 Compare July 30, 2020 03:54
@stevengum stevengum requested a review from allenjzhang as a code owner July 30, 2020 03:54
@openapi-sdkautomation openapi-sdkautomation bot force-pushed the sdkAutomation/Microsoft.Azure.Management.BotService branch from b02a6c1 to fa07172 Compare July 30, 2020 10:01
@openapi-sdkautomation openapi-sdkautomation bot force-pushed the sdkAutomation/Microsoft.Azure.Management.BotService branch from fa07172 to efb4379 Compare July 30, 2020 11:10
@isra-fel
Copy link
Member

Hi @stevengum

You are free to base you work on the auto-generated branch, but unlike other SDKs, .NET SDK requires some manual work on top of generated code, like updating versions, release notes, and most importantly tests, so we cannot just simply merge from an automation branch.

Please refer to https://github.com/Azure/adx-documentation-pr/blob/master/engineering/adx_netsdk_process.md#sdk-generation-from-updated-spec for the detailed steps. Thanks

@isra-fel isra-fel added Mgmt This issue is related to a management package. needs-revision labels Aug 14, 2020
@isra-fel
Copy link
Member

Closing the PR for lack of activity. Feel free to reopen to continue working. Thanks

@isra-fel isra-fel closed this Aug 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Mgmt This issue is related to a management package.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants