-
Notifications
You must be signed in to change notification settings - Fork 62
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
ci: Enable ROOT6 tests #452
Conversation
997eab3
to
6af042c
Compare
e52167e
to
ba614cb
Compare
ba614cb
to
205f44a
Compare
205f44a
to
e913c17
Compare
See discussion in #452 #452 (review)
e8fcfa2
to
c62e563
Compare
e96c8ef
to
fb05a76
Compare
fb05a76
to
9424037
Compare
This one is ready to go. Looks like all of the ROOT6 tests we have set up in CI pass. Only the usual checks are performed, i.e. the bfc.C macro can run on small real and simulated samples from various years without checking the quality of the output. Let me know if more inputs and options should be checked this way. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do I understand it correctly that all these includes were not needed with ROOT5 and now they are needed in ROOT6?
@iraklic Yes, rootcint is more strict in that sense. It needs to see the definitions of all used types while processing the macros |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Happy to see this milestone reached
Co-authored-by: Dmitry Kalinkin <dmitry.kalinkin@gmail.com>
No description provided.