Skip to content

feat: considering additionalProperties missing default to false - #59

Merged
vmarchaud merged 6 commits into
Eywek:mainfrom
Tchoupinax:feature/additionnal-properties
Jan 27, 2026
Merged

feat: considering additionalProperties missing default to false#59
vmarchaud merged 6 commits into
Eywek:mainfrom
Tchoupinax:feature/additionnal-properties

Conversation

@Tchoupinax

@Tchoupinax Tchoupinax commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Adding the log/throw feature on additional properties is not very useful if you do not have this field for every property in your schema.

This pull requests patches the library:

  • Add additionalproperties: false on every object by default it additionalproperties is not provided
  • Add a new jsdoc property additionalproperties: _<boolean>_ that allows to specify the value

@Tchoupinax
Tchoupinax force-pushed the feature/additionnal-properties branch from 0795bc6 to 5c8cb54 Compare January 26, 2026 18:50
@Tchoupinax
Tchoupinax marked this pull request as draft January 26, 2026 20:14
@Tchoupinax
Tchoupinax force-pushed the feature/additionnal-properties branch from 90782e8 to 59d6b51 Compare January 26, 2026 21:02
@Tchoupinax
Tchoupinax marked this pull request as ready for review January 27, 2026 08:54
@vmarchaud
vmarchaud merged commit de02552 into Eywek:main Jan 27, 2026
2 checks passed
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.

2 participants