Code review comment for ~ilasc/launchpad:webhook-livefs

Revision history for this message
Colin Watson (cjwatson) wrote :

This mostly LGTM, thanks! I had a few minor comments from a line-by-line review, but the "needs fixing" is mainly for correcting the feature flag.

(As discussed on IRC, the test failure was in fact from my fix-webhook-visibility-check branch that's a prerequisite of this one. Sorry about that.)

review: Needs Fixing

« Back to merge proposal