-
Notifications
You must be signed in to change notification settings - Fork 43
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
Wrong client name #1359
Comments
The names aren't wrong per se as they come from the hierarchical client specified in the tsp (the names aren't great though). Can you please clarify what was the expectation? |
Thanks for clarifying. Are we sure that the tsp is matching the OpenAPI? I suspect we're going to see this pattern a lot. So, before we go and modify the emitter, I'd like to confirm that the tsp matches the swagger. |
CC @tadelesh |
Hmm, maybe it's the case that since the emitter turns off hierarchical clients for ARM, we also turn off hierarchical client naming for ARM too (assuming the tsp is correct). |
is that name from |
Correct. |
tsp: https://github.com/Azure/azure-rest-api-specs/blob/main/specification/communitytraining/Community.Management/main.tsp
The text was updated successfully, but these errors were encountered: