Code review comment for ~xavpaice/charm-hw-health:lp1855095

Revision history for this message
Paul Goins (vultaire) wrote :

I think this basically works.

Arguably I'd suggest simply removing the lines and adding them back in in the future if support is ever properly added, so as to remove cruft. It depends on how likely we think that support will be added in the future.

It might be worth adding a comment to the code with an explanation, but at the same time, "git blame" can be used to find the commit that these were commented in, and the commit message can provide this explanation.

However, I would strongly suggest that we mention lp#1855095 either in the commit message or in a comment, as that won't be readily viewable via git after this gets merged.

review: Needs Fixing

« Back to merge proposal