-
Notifications
You must be signed in to change notification settings - Fork 157
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
refactor!: Split helper header into multiple files #2153
Conversation
Excellent initiative! |
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.
Looks good, ok to merge if it compiles.
Is this breaking? I guess so?
Codecov Report
@@ Coverage Diff @@
## main #2153 +/- ##
=======================================
Coverage 49.50% 49.51%
=======================================
Files 436 439 +3
Lines 25128 25128
Branches 11605 11603 -2
=======================================
+ Hits 12440 12441 +1
Misses 4431 4431
+ Partials 8257 8256 -1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
📊 Physics performance monitoring for d9374fcSummary VertexingSeedingCKFAmbiguity resolutionTruth tracking (Kalman Filter)Truth tracking (GSF) |
CI is passing now - could you re-approve? @paulgessinger @asalzburger |
In #2151 I discovered that modifying
Helpers.hpp
basically triggers a full rebuild and that we put a bunch of different stuff into it. In this PR I try to improve things by splitting upHelpers.hpp
into multiple headers