Skip to content
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

Migrate to 13.20.0 #89

Merged
merged 43 commits into from
Nov 16, 2023
Merged

Migrate to 13.20.0 #89

merged 43 commits into from
Nov 16, 2023

Conversation

shuruev
Copy link

@shuruev shuruev commented Nov 16, 2023

The 13.20.0 version was materialized in this branch:
https://github.com/servicetitan/NSwag/tree/fork/release-13-20-0

Our patched code remains the same.
Also upgraded Newtonsoft.Json to 13.0.3.

RicoSuter and others added 30 commits November 24, 2022 20:37
…v2 (RicoSuter#4314)

* Remove description from OpenApiResponse when serializing for Swagger v2 to prevent error "Structural error at paths.somepath.get.responses.304 should NOT have additional properties additionalProperty: description" [PR].

* Remove description from OpenApiResponse when serializing for Swagger v2 to prevent error "Structural error at paths.somepath.get.responses.304 should NOT have additional properties additionalProperty: description" [PR].
OpenApi which contains deprecated schemas correctly generates `[Obsolete]` attribute. If the API document does not propagate `deprecate` all the way to path/method resulting c# code produce warning CS0612: '...' is obsolete
Fix Newtonsoft.Json being used instead of System.Text.Json
…coSuter#4388)

Duplicate response code in XML documentation response xml tag causes
somewhat hard to debug exception, which is now made more descriptive.

Did not yet figure out how to make an unit test for this one or if some
other other exception type would be more suitable than
InvalidOperationException.

Bug: Duplicate return code declaration causes nasty exception
id: RicoSuter#2863

Co-authored-by: Antti Viita <antti.viita@vividworks.com>
* add axios abort signal

* unit test
* Unwrap ValueTask<T> return types, same as Task<T>, unwrap (Value)Task<ActionResult<T>> and unify unwrapping for consistency RicoSuter#4373

* lahma suggestion for performance
This reverts commit 21bf926.
Rico Suter and others added 12 commits May 4, 2023 00:13
…r" in generated code (RicoSuter#4422)

* Disable warning CS8604 "Possible null reference argument for parameter" in code generation file header template

* Restore warning in file footer
* Update Client.Class.liquid

fix RicoSuter#4466

* Update Client.Class.liquid

---------

Co-authored-by: Rico Suter <mail@rsuter.com>
@shuruev shuruev requested a review from SergeiPavlov November 16, 2023 06:24
@shuruev shuruev self-assigned this Nov 16, 2023
@shuruev shuruev merged commit 459583a into master Nov 16, 2023
1 check failed
@shuruev shuruev deleted the dev/migrate-to-13-20-0 branch November 16, 2023 06:29
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.