Code review comment for lp://staging/~allanlesage/uci-engine/coverage-extractor

Revision history for this message
Francis Ginther (fginther) wrote :

Allan, I'm fine with a follow up MP. Also, I may have crossed my comments with your responses. As this component isn't deployed just yet. I'm ok to just address the TODO and XXX comments before top-approving. Please ping me tomorrow (or is it now today) if there is any confusion or my inline comments are not visible.

review: Needs Fixing

« Back to merge proposal