Skip to content

emit "additionalProperties": false with rfl::NoExtraFields - #728

Merged
liuzicheng1987 merged 2 commits into
munich-data-engineering:mainfrom
gg-sr:schema-additional-properties
Oct 26, 2026
Merged

liuzicheng1987 merged 2 commits into
munich-data-engineering:mainfrom
gg-sr:schema-additional-properties

Conversation

@gg-sr

@gg-sr gg-sr commented Sep 23, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@gg-sr
gg-sr force-pushed the schema-additional-properties branch from 30be66a to 2bcf820 Compare September 23, 2026 08:08
@liuzicheng1987

Copy link
Copy Markdown
Collaborator

@gg-sr I think this makes a lot of sense, but the tests need to be adapted to make this work. Could you take care of that?

@gg-sr
gg-sr force-pushed the schema-additional-properties branch from 88658db to bc99d8c Compare September 27, 2026 23:13
@gg-sr

gg-sr commented Sep 28, 2026

Copy link
Copy Markdown
Contributor Author

@liuzicheng1987 Right, fixed in bc99d8c, sorry!

One of the CI checks failed, but it seems like a transient error. Some checks took a while to finish, but I don't think it's related to this change.

@Altren

Altren commented Oct 26, 2026

Copy link
Copy Markdown
Contributor

@liuzicheng1987 The only CI failure here is vcpkg hiccup, I think it should be merged if that was the only blocker

@liuzicheng1987

Copy link
Copy Markdown
Collaborator

@Altren, indeed, you are right.
@gg-sr thanks for the contribution and sorry it took me so long to merge.

@liuzicheng1987
liuzicheng1987 merged commit 587b72e into munich-data-engineering:main Oct 26, 2026
175 of 176 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.

3 participants