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
Introduce Config interfaces #4598
Introduce Config interfaces #4598
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4598 +/- ##
=======================================
Coverage 73.40% 73.41%
=======================================
Files 345 345
Lines 13739 13739
=======================================
+ Hits 10085 10086 +1
+ Misses 3021 3020 -1
Partials 633 633
Continue to review full report at Codecov.
|
a4e0681
to
dc08f63
Compare
dc08f63
to
6ce9f1a
Compare
6ce9f1a
to
c43fb29
Compare
c43fb29
to
94b4489
Compare
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 great to me!
Let me remove those wip commits |
94b4489
to
1f577f9
Compare
BTW we only squash merge now to not have to take care of these commit cleanups anymore. |
Signed-off-by: David Gageot <david@gageot.net>
1f577f9
to
d914c30
Compare
This is an additional non-dangerous step towards getting rid of RunContext as much as possible.
It introduces a
Config
interface for each constructor that used to depend on a RunContext. This acts as a "facade" on the RunContext and can only access information by their getter, making them more read-only. This break some hard coupling that we have which is also nice.Signed-off-by: David Gageot david@gageot.net