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

Autogenerate the SDK from the OpenAPI spec (3.0.0) #102

Open
wants to merge 46 commits into
base: main
Choose a base branch
from
Open

Conversation

AndrewBenzSW
Copy link
Contributor

This PR will re-add the new SDK as a 3.0.0-beta version. A few changes since the previous PR are:

  • Enable the nullableReferenceTypes flag when generating the api
  • Use DateTimeOffset instead of DateTime in the methods and models
  • Update the Github action so that packages can be manually published as long as the version includes a pre-release tag

AndrewBenzSW and others added 30 commits July 16, 2024 08:22
Previously, the SDK would throw an exception when trying to parse an empty string.
Now we will return the empty string, if a string response is requested.

If the caller was expecting a deserialized type, an exception will still be raised for no content.
Including the version in every file causes every file to be modified
any time the generator is run. This creates a large diff that obscures the actual changes.
The version is already known in the generation/swagger.json file.

Add warning that file is generated and should not be modified.
Convert doubles to decimals
@AndrewBenzSW AndrewBenzSW marked this pull request as ready for review February 5, 2025 15:14
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