Skip to content

fix: retain unknown keys in options (incl. overrides)#91

Merged
43081j merged 1 commit into
mainfrom
jg/retain-overrides
Jun 3, 2026
Merged

fix: retain unknown keys in options (incl. overrides)#91
43081j merged 1 commit into
mainfrom
jg/retain-overrides

Conversation

@43081j
Copy link
Copy Markdown
Collaborator

@43081j 43081j commented Jun 3, 2026

An alternative to #73.

Closes #70.

This updates normalizeFormatOptions to iterate over own keys rather
than specific in checks. This way, we retain extra keys while still
validating the known ones.

Doing it here means overrides and configs alike both retain keys.

cc @fisker

An alternative to #73.

Closes #70.

This updates `normalizeFormatOptions` to iterate over own keys rather
than specific `in` checks. This way, we retain extra keys while still
validating the known ones.

Doing it here means overrides and configs alike both retain keys.
@43081j 43081j requested a review from fisker June 3, 2026 09:23
@43081j 43081j changed the title fix: retain unknown keys in overrides fix: retain unknown keys in options (incl. overrides) Jun 3, 2026
@43081j 43081j merged commit 114c874 into main Jun 3, 2026
4 checks passed
@43081j 43081j deleted the jg/retain-overrides branch June 3, 2026 11:48
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.

Support Configuration Overrides for the experimental cli.

2 participants