-
-
Notifications
You must be signed in to change notification settings - Fork 6.9k
[aspnetcore] Support async tasks and some code cleanups #2629
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
Conversation
…nerate wwwroot and dont make model class partial and default to no swashbuckle, , workarodun empty string cliOption
👍 Thanks for opening this issue! The team will review the labels and make any necessary changes. |
...napi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java
Outdated
Show resolved
Hide resolved
...napi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java
Outdated
Show resolved
Hide resolved
...napi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java
Outdated
Show resolved
Hide resolved
...napi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java
Show resolved
Hide resolved
...napi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java
Show resolved
Hide resolved
Sorry I hadn't seen this PR, been pretty busy the past couple of weeks. Aside from a handful of comments, this looks good. It seems almost identical to the previous work but with some cleanup to cli switches. |
Thanks for the update. Looks like all my comments were addressed. @wing328 mind taking a quick look and merging? |
...napi-generator/src/main/java/org/openapitools/codegen/languages/AspNetCoreServerCodegen.java
Show resolved
Hide resolved
* master: (40 commits) Remove quotation marks around {{paramName}} for header params in api-body.mustache (#2727) Add FiNC Technologies (#2728) fix missing parenthesis for http bearer auth (#2723) Add missing closing parenthesis (#2720) update perl test with correct body parameter (#2717) [Java][Spring] Fix template for reactive implementation with `interfaceOnly` parameter (#2437) Bugfix(Perl): Support nested primitive types in ARRARY or HASH for basic object (#2713) Remove `-XX:MaxPermSize` (#2712) Remove setting generateAliasAsModel in rust server generator (#2714) update rust server samples Revert "update rust samples" update rust samples update samples [Rust Server] Improve XML support (#2504) Improve CONTRIBUTING.md (#2699) [PHP][Lumen] Rename template folder (#2707) [aspnetcore] Support async tasks and some code cleanups (#2629) [C++][Pistache] Fixed #2643 (#2653) update petstore samples (#2697) [JAVA][Webclient]fix select body for url encoded media type. (#2686) ...
PR checklist
./bin/
to update Petstore sample so that CIs can verify the change. (For instance, only need to run./bin/{LANG}-petstore.sh
,./bin/openapi3/{LANG}-petstore.sh
if updating the {LANG} (e.g. php, ruby, python, etc) code generator or {LANG} client's mustache templates). Windows batch files can be found in.\bin\windows\
.master
,. Default:3.4.x
,4.0.x
master
.Description of the PR
Support for async and Task for both library and program
Support for Controller and Model namespaces.
Small fixes and code generation clean up