Skip to content

Conversation

@knom
Copy link
Contributor

@knom knom commented Oct 14, 2019

PR checklist

  • Read the contribution guidelines.
  • Ran the shell script under ./bin/ to update Petstore sample so that CIs can verify the change. (For instance, only need to run ./bin/{LANG}-petstore.sh and ./bin/security/{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\.
  • Filed the PR against the correct branch: 3.0.0 branch for changes related to OpenAPI spec 3.0. Default: master.
  • Copied the technical committee to review the pull request if your PR is targeting a particular programming language.

Description of the PR

@mandrean - I've added support for .NET Core 3 / Swashbuckle 5.
#9781

@frantuma
Copy link
Contributor

Thanks for this! we will review soon.

@knom
Copy link
Contributor Author

knom commented Dec 4, 2019

@frantuma any update?

@knom
Copy link
Contributor Author

knom commented Dec 12, 2019

Hmm.. I really liked the project - and did a few contributions every now and then..
(especially in combination w. https://editor.swagger.io)

But the community has changed, @wing328 is gone it seems..
Not sure how active / alive the codegen still is.. looking at alternatives right now!

@HugoMario
Copy link
Contributor

thanks for your PR @knom, and sorry for this delay. Next time you need help with something in this project. Please, let me know.

@HugoMario HugoMario merged commit 9967374 into swagger-api:master Feb 9, 2020
@rdecarreau
Copy link

Was this built into the CLI? I have 3.0.18 CLI and it only builds aspnet2.2.

@HugoMario
Copy link
Contributor

@rdecarreau this was written for version 2, but will ported soon to codegen v3

@phil-warner
Copy link

Hi @HugoMario was this ported to codegen v3 yet? I'll have to drop to version 2 if not.

@HugoMario
Copy link
Contributor

This was referenced Mar 11, 2021
This was referenced Mar 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants