cxx-qt-gen: add support for passing in CfgEvaluator #1168
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1168 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 71 73 +2
Lines 11882 12426 +544
==========================================
+ Hits 11882 12426 +544 ☔ View full report in Codecov by Sentry. |
ahayzen-kdab
force-pushed
the
1133-cfg-evaluator-2
branch
2 times, most recently
from
9e223b7 to
3fe67ce
Compare
ahayzen-kdab
force-pushed
the
1133-cfg-evaluator-2
branch
from
3fe67ce to
1673abc
Compare
LeonMatthesKDAB
left a comment
LeonMatthesKDAB
left a comment
There was a problem hiding this comment.
Great work overall.
I thinkg this is lacking a snapshot test though, so it's easier to see where cfg attributes are now enabled and how that changes the generation.
Also, cfg attributes on QObject/QEnum don't seem to be respected yet.
We're also missing a CHANGELOG entry 😉
ahayzen-kdab
force-pushed
the
1133-cfg-evaluator-2
branch
5 times, most recently
from
9359ce5 to
fe7f8a2
Compare
LeonMatthesKDAB
left a comment
LeonMatthesKDAB
left a comment
There was a problem hiding this comment.
The test output looks pretty good.
Enabled items on disabled types may still be an issue.
They will fail to compile, which is expected, but the error messages are likely to be cryptic.
If there is an easy way to give a nicer error message we should probably do so.
However, if it's hard to give a good error message, we can keep it as-is.
And may confuse the results.
This avoids collisions between paths, we were seeing odd issues around some files having a different tracked amount in codecov vs lcov. It appears to be due to having coverage results from cxx_gen with the same file paths. We do not want coverage data for anything else anyway so filter them out.
ahayzen-kdab
force-pushed
the
1133-cfg-evaluator-2
branch
from
fe7f8a2 to
69cd338
Compare
ahayzen-kdab
force-pushed
the
1133-cfg-evaluator-2
branch
from
69cd338 to
729a56e
Compare
2 participants
No description provided.