Tarmac

Code review comment for lp:~mars/tarmac/fix-merge-error-updates-proposals

  1. fix-merge-error-updates-proposals
  2. Merge into lp-tarmac
Revision history for this message
Diogo Matsubara (matsubara) wrote :

Hi Mars,

I have a small comment about some commented out code on lines 39 and 40. It looks like you commented out the code but the intention was to remove it.

Another thing is that you define the MergeErrorInjector twice in test code. Is there a reason for that?

Other than that, it looks good. Thanks!

« Back to merge proposal