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

Add support for additionalProperties: false on empty objects #555

Closed
wants to merge 1 commit into from
Closed

Add support for additionalProperties: false on empty objects #555

wants to merge 1 commit into from

Conversation

tdeo
Copy link

@tdeo tdeo commented Jul 17, 2023

Fix #525

For a schema with:

type: object
additionalProperties: false

The generated TS definition should be:

{
  [key: string]: never
}

to ensure that the object contains no properties at all

@coveralls
Copy link

Coverage Status

coverage: 91.035% (+0.02%) from 91.02% when pulling 0bee8e5 on tdeo:td/support_false_additional_properties_on_empty_object into 24d8037 on horiuchi:master.

@tdeo tdeo closed this Oct 17, 2023
@tdeo tdeo deleted the td/support_false_additional_properties_on_empty_object branch October 17, 2023 13:09
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.

AdditionalProperties in swagger definition creates broken TS interface
2 participants