openapi-typescript icon indicating copy to clipboard operation
openapi-typescript copied to clipboard

Don't remove `null` type if a default is present.

Open luhn opened this issue 10 months ago • 5 comments

Changes

Fixes #2144

Fixes #2055

How to Review

Pretty straightforward PR

Checklist

  • [x] Unit tests updated
  • [ ] docs/ updated (if necessary)
  • [x] pnpm run update:examples run (only applicable for openapi-typescript)

luhn avatar Feb 07 '25 06:02 luhn

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...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

netlify[bot] avatar Feb 07 '25 06:02 netlify[bot]

⚠️ 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

changeset-bot[bot] avatar Feb 07 '25 06:02 changeset-bot[bot]

This is nice, can we merge it @htunnicliff?

ben-xD avatar Feb 24 '25 12:02 ben-xD

+1

danielkv avatar Feb 28 '25 14:02 danielkv

The maintainers team is looking at this issue and its corresponding PR. Please refrain from adding +1 comments.

duncanbeevers avatar Feb 28 '25 17:02 duncanbeevers

Released in 7.7.2!

drwpow avatar May 10 '25 14:05 drwpow