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

groot: add handling of top-level key with string payload #1009

Merged
merged 2 commits into from
Mar 5, 2024

Conversation

sbinet
Copy link
Member

@sbinet sbinet commented Mar 5, 2024

No description provided.

@sbinet sbinet force-pushed the groot-top-level-string branch 2 times, most recently from c99a2ee to 1cae426 Compare March 5, 2024 11:40
@codecov-commenter
Copy link

codecov-commenter commented Mar 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 67.25%. Comparing base (ac11738) to head (9004c7e).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1009      +/-   ##
==========================================
+ Coverage   67.23%   67.25%   +0.02%     
==========================================
  Files         491      492       +1     
  Lines       71877    71909      +32     
==========================================
+ Hits        48328    48365      +37     
+ Misses      21221    21213       -8     
- Partials     2328     2331       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sbinet sbinet force-pushed the groot-top-level-string branch from 1cae426 to 8900ca8 Compare March 5, 2024 13:20
Signed-off-by: Sebastien Binet <binet@cern.ch>
@sbinet sbinet force-pushed the groot-top-level-string branch from 8900ca8 to 9004c7e Compare March 5, 2024 13:51
Signed-off-by: Sebastien Binet <binet@cern.ch>
@sbinet sbinet merged commit c104150 into go-hep:main Mar 5, 2024
6 checks passed
@sbinet sbinet deleted the groot-top-level-string branch March 5, 2024 14:46
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