Code review comment for lp:~rockstar/launchpad/merge-queues-model

Revision history for this message
Aaron Bentley (abentley) wrote :

Please add an extra blank line before the definition of InvalidMergeQueueConfig, and make BranchMergeQueue.new a classmethod, not a staticmethod, so you don't need to repeat the class name.

Otherwise, this is good.

review: Approve

« Back to merge proposal