-
Notifications
You must be signed in to change notification settings - Fork 45
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
azure-http-specs, add clientNamespace #1929
base: main
Are you sure you want to change the base?
azure-http-specs, add clientNamespace #1929
Conversation
All changed packages have been documented.
|
You can try these changes here
|
name: "ClientNamespaceFirstClient", | ||
service: Client.ClientNamespace, | ||
}) | ||
@clientNamespace("client.clientnamespace", "java") |
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.
is it possible to remove the scope? i believe every language needs to test ns customization.
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.
Every language would need to use slight different SDK namespace. E.g. csharp would use maybe Client.ClientNamespace
.
And for Azure, .NET maybe Azure.###
, and Java be com.azure.###
. They are very different.
So I think whenever we had @clientNamespace
, it would be one line for one language. I expect other language fill this line, when they are implementing the clientNamespace feature.
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.
@msyyc could you help to add python part?
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.
updated for python.
fix Azure/cadl-ranch#758
move from Azure/cadl-ranch#775
This scenario mainly test the case when
@clientNamespace
applied to TypeSpec Namespace/Interface/Model/Enum.So far it does not test corner cases, like client/subclient in different namespace, name conflict for model in different namespace, etc.
Current cases include