Don't remove `null` type if a default is present.
Changes
Fixes #2144
Fixes #2055
How to Review
Pretty straightforward PR
Checklist
- [x] Unit tests updated
- [ ]
docs/updated (if necessary) - [x]
pnpm run update:examplesrun (only applicable for openapi-typescript)
Deploy Preview for openapi-ts ready!
| Name | Link |
|---|---|
| Latest commit | 705f3e458dcc9ed598e1a1bfc7d2d2310fab0de4 |
| Latest deploy log | https://app.netlify.com/projects/openapi-ts/deploys/67a5ad4ce3812f0008df4b8b |
| Deploy Preview | https://deploy-preview-2145--openapi-ts.netlify.app |
| Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
To edit notification comments on pull requests, go to your Netlify project configuration.
⚠️ No Changeset found
Latest commit: 705f3e458dcc9ed598e1a1bfc7d2d2310fab0de4
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
This PR includes no changesets
When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
This is nice, can we merge it @htunnicliff?
+1
The maintainers team is looking at this issue and its corresponding PR. Please refrain from adding +1 comments.
Released in 7.7.2!