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

Updating namespace names as per the review #31206

Merged

Conversation

bsomeshekar
Copy link
Contributor

Updating namespace names as per the review
https://github.com/Azure/azure-sdk-pr/issues/1889

Choose a PR Template

Switch to "Preview" on this description then select one of the choices below.

Click here to open a PR for a Data Plane API.

Click here to open a PR for a Control Plane (ARM) API.

Copy link

openapi-pipeline-app bot commented Oct 24, 2024

Next Steps to Merge

✅ All automated merging requirements have been met! To get your PR merged, see aka.ms/azsdk/specreview/merge.

Copy link

openapi-pipeline-app bot commented Oct 24, 2024

Generated ApiView

Language Package Name ApiView Link
Go sdk/resourcemanager/commonedgesitemanageroperations/armcommonedgesitemanageroperations https://apiview.dev/Assemblies/Review/2ab0fefa2b784af0a4500f886f513585?revisionId=feb5aa8a965f4b12b1c4f53522747624
Go sdk/resourcemanager/sitemanager/armsitemanager https://apiview.dev/Assemblies/Review/72a9a19f642a4523baaabef4cf6166cc?revisionId=460a19c662124846ae562600254245aa
.Net Azure.ResourceManager.CommonEdgeSiteManagerOperations https://apiview.dev/Assemblies/Review/d0ddeae889ce49e0a057706d7fb7d2ab?revisionId=eb0b42ab21564a44b4c614127fbbdf43
.Net Azure.ResourceManager.SiteManager https://apiview.dev/Assemblies/Review/4443b818f7fc4e9385b1475b2d87e50b?revisionId=4db06a64ae76483398deb1ab16600472
Java azure-resourcemanager-commonedgesitemanageroperations https://apiview.dev/Assemblies/Review/45fb7269d2aa4a909b68a9fc4dc2d00c?revisionId=95e09d85c3da41e79b118fc948ee9883
Java azure-resourcemanager-edgemicrosoftedgeedge https://apiview.dev/Assemblies/Review/9c5cc1959d294bbf9c857897845dcc46?revisionId=372c3c99197947c6acd00a5fc16d94ba
Java azure-resourcemanager-edgemicrosoftedgesites https://apiview.dev/Assemblies/Review/ffb8803cb6194bdc98fed101921ccd90?revisionId=3a87f04b65e34cd8ada6b1615913835d
Java azure-resourcemanager-sitemanager https://apiview.dev/Assemblies/Review/5e68408b1f514ec0ba387f32eb2857b1?revisionId=25ff5e1111af4f059fbee4d6cca73624
JavaScript @azure/arm-commonedgesitemanageroperations https://apiview.dev/Assemblies/Review/cb53d59f55844d74a3124e21848a85a7?revisionId=94902d920dd0409cbd8843211fd48629
JavaScript @azure/arm-sitemanager https://apiview.dev/Assemblies/Review/932a5d7082944042a11a50d2ec27b68d?revisionId=2aa2edc161304aec85eadcb59358467d
Python azure-mgmt-commonedgesitemanageroperations https://apiview.dev/Assemblies/Review/d4653c455e944e5b846588fcdf770ee8?revisionId=75cb3976116146d6beadef5e668a3f01
Python azure-mgmt-sitemanager https://apiview.dev/Assemblies/Review/f1dabe90a7064cdbb4d5f8f89a1f182b?revisionId=4c5a50c3a1894b27b0a2839038c0f655

@AzureRestAPISpecReview AzureRestAPISpecReview added ARMReview ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test resource-manager RPaaS WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Oct 24, 2024
@bsomeshekar bsomeshekar added the PublishToCustomers Acknowledgement the changes will be published to Azure customers. label Oct 24, 2024
@bsomeshekar bsomeshekar removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Oct 24, 2024
@AzureRestAPISpecReview AzureRestAPISpecReview added TypeSpec Authored with TypeSpec WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required labels Oct 24, 2024
@ramoka178 ramoka178 added the ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review label Oct 24, 2024
@openapi-pipeline-app openapi-pipeline-app bot removed the WaitForARMFeedback <valid label in PR review process> add this label when ARM review is required label Oct 24, 2024
Copy link
Member

@msyyc msyyc left a comment

Choose a reason for hiding this comment

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

Approve for Python and @bsomeshekar you also need approval from other language owners.

…eshekar/azure-rest-api-specs into basavaraj-edgesites-updatingsdkname
@tadelesh
Copy link
Member

Approved for Go.

@bsomeshekar bsomeshekar enabled auto-merge (squash) October 30, 2024 09:41
@bsomeshekar
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@bsomeshekar bsomeshekar removed the request for review from jsntcy November 4, 2024 06:29
@bsomeshekar bsomeshekar merged commit 03bc5a3 into Azure:main Nov 4, 2024
29 of 30 checks passed
haagha pushed a commit that referenced this pull request Nov 19, 2024
* Updating namespace names as per the review
Azure/azure-sdk-pr#1889

* Setting namespace for java sdk

* Fixing pipeline errors

* Adding emitter for python

* Fixing pipeline errors

* Adding service-dir to fix pipeline error

* Fixing pipeline errors

* Revert "Fixing pipeline errors"

This reverts commit f638130.

* Adding configs in typescript file

* update

* disable sample generation for csharp

* disable sample generation of csharp for sitemanager

* Removing emitter-output-dir based on review comments

---------

Co-authored-by: Yuchao Yan <[email protected]>
Co-authored-by: kazrael2119 <[email protected]>
Co-authored-by: Wei Hu <[email protected]>
Co-authored-by: Wei Hu <[email protected]>
digvijaymali94 pushed a commit to digvijaymali94/azure-rest-api-specs that referenced this pull request Dec 17, 2024
* Updating namespace names as per the review
https://github.com/Azure/azure-sdk-pr/issues/1889

* Setting namespace for java sdk

* Fixing pipeline errors

* Adding emitter for python

* Fixing pipeline errors

* Adding service-dir to fix pipeline error

* Fixing pipeline errors

* Revert "Fixing pipeline errors"

This reverts commit f638130.

* Adding configs in typescript file

* update

* disable sample generation for csharp

* disable sample generation of csharp for sitemanager

* Removing emitter-output-dir based on review comments

---------

Co-authored-by: Yuchao Yan <[email protected]>
Co-authored-by: kazrael2119 <[email protected]>
Co-authored-by: Wei Hu <[email protected]>
Co-authored-by: Wei Hu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review PublishToCustomers Acknowledgement the changes will be published to Azure customers. ReadyForApiTest <valid label in PR review process>add this label when swagger and service APIs are ready for test resource-manager RPaaS TypeSpec Authored with TypeSpec
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants