Skip to content

Add test for feature flags - #58

Merged
jblomer merged 1 commit into
root-project:mainfrom
jblomer:add-feature-flag
Jan 23, 2026
Merged

Add test for feature flags#58
jblomer merged 1 commit into
root-project:mainfrom
jblomer:add-feature-flag

Conversation

@jblomer

@jblomer jblomer commented Jan 20, 2026

Copy link
Copy Markdown
Contributor

Partially fixes #34

@jblomer jblomer self-assigned this Jan 20, 2026
Comment thread structure/feature_flag/write.C
Comment thread structure/feature_flag/write.C
@hahnjo hahnjo added this to the 1.0 milestone Jan 22, 2026

@hahnjo hahnjo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test using Internal functions LGTM, one comment about the JSON output

Comment thread structure/read_structure.hxx
Comment thread structure/read_structure.hxx Outdated
@jblomer
jblomer requested a review from hahnjo January 22, 2026 14:53
@jblomer
jblomer merged commit c42737f into root-project:main Jan 23, 2026
1 check passed
@jblomer
jblomer deleted the add-feature-flag branch January 23, 2026 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test for RNTuple feature flags

3 participants