Azure / azure-sdk-for-net

This repository is for active development of the Azure SDK for .NET. For consumers of the SDK we recommend visiting our public developer docs at https://learn.microsoft.com/dotnet/azure/ or our versioned developer docs at https://azure.github.io/azure-sdk-for-net.
MIT License
5.26k stars 4.6k forks source link

[Text Analytics] Revisit Legacy Generator Customizations #28258

Closed jsquire closed 5 months ago

jsquire commented 2 years ago

Summary

Because the Text Analytics library supports two REST services as its backend, it makes use of two generated layers - one of which is associated with the legacy Text Analytics REST service. The generated types for the legacy service are intended to be fully internal and should not appear as part of the public API surface for the client library. To control visibility, a set of transforms is configured for the AutoRest generator.

However, some types are not honoring the internal transformation. Of these, there are several enum types used as parameters for request bodies and cannot be designated as partial to allow more fine-grained generator overrides at the code level. Currently, these have been fully shadowed as a custom class to control visibility, which requires that their members be kept in-sync with the service definition manually.

Scope of Work

Success Criteria

--EITHER--

--OR--

References and Resources

github-actions[bot] commented 9 months ago

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @assafi.

github-actions[bot] commented 5 months ago

Hi @jsquire, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.