-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[contoso] Move TSP for folder structure v2 #34679
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
base: main
Are you sure you want to change the base?
Conversation
Next Steps to MergeNext steps that must be taken to merge this PR:
|
PR validation pipeline restarted successfully. If there is ApiView generated, it will be updated in this comment. |
API Change CheckAPIView identified API level changes in this PR and created the following API reviews
|
…rest-api-specs into contoso-tsp-v2
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.
The folder structure should be identical to what you did in https://github.com/Azure/azure-rest-api-specs/pull/34677/files
It's just that the tspconfig.yaml, TS files & examples folders also go in the ServiceName folders
I was intentionally splitting the changes to TSP and changes to readme/swagger into separate PRs for analysis. I can create a third PR with both changes merged. |
Changelog
contosowidgetmanager/Contoso.WidgetManager
tocontosowidgetmanager/data-plane/Azure.Contoso.WidgetManager
.contosowidgetmanager/Contoso.Management
tocontosowidgetmanager/resource-manager/Microsoft.Contoso
.contosowidgetmanager/Contoso.WidgetManager.Shared/main.tsp
toshared.tsp
in bothdata-plane
andresource-manager
folders.emitter-output-dir
intspconfig.yaml
files for new folder deptharm-types-dir
inresource-manager/.../tspconfig.yaml
for new folder depthNote: Ignore temp infra changes outside folder
specification/contosowidgetmanager