-
Notifications
You must be signed in to change notification settings - Fork 25.3k
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
Add documentation to support code generator MSBuild #8461
Comments
@scottaddie is any of this completed? |
@Rick-Anderson No. Looks like the engineering work is still in progress. I've updated Doug's description with the updated issue. |
@glennc given the low number of code generators, how important is this documentation for our partners i.e. is it really a P1? |
Reframed to more closely align with #8465 |
Discussed offline w/ @glennc. This issue should be addressed for the benefit of the ecosystem, including partners who choose not to interact with Microsoft directly. |
General
dotnet/aspnetcore#4896 includes a number of extension points for code and document generator providers. We plan to submit example extensions to the https://github.com/RSutor/NSwag repo but not to do the same for (say) Swashbuckle or AutoRest. Owners of other generators will thus require documentation of how to work with the coming Microsoft.Extensions.ApiDescription.Client package.
Requests for new Topics
Topic will cover requirements for implementing
BlahDocumentGenerator
,BlahCSharpCodeGenerator
andBlahTypeScriptCodeGenerator
targets, how those targets will be invoked, and suggested package dependencies. See RicoSuter/NSwag#1587 for example; that issue covers creating a package containingNSwagCSharpCodeGenerator
andNSwagTypeScriptCodeGenerator
targets. (RicoSuter/NSwag#1588 covers implementing theBlahDocumentGenerator
target but the pattern is identical.)Contents
Placement
This documentation doesn't really belong in the "Building Web APIs" tutorial and it's not clear to me where those wishing to extend ASP.NET Core would look first. Perhaps we need a new "chapter" providing guidance for those extending the framework?
CC
@glennc
The text was updated successfully, but these errors were encountered: