Code review comment for ~ilasc/launchpad:eslint-fix-no-new

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

Generally LGTM, but could you use `eslint-disable-next-line` rather than `eslint-disable`? As I understand it, `eslint-disable` disables the rule for the rest of the block or until a matching `eslint-enable` is reached, and we don't need to be that extensive here.

review: Approve

« Back to merge proposal