Code review comment for lp://qastaging/~jaypipes/drizzle/publisher

Revision history for this message
Brian Aker (brianaker) wrote :

Hi!

Until it fits under the test framework we have it won't be mergable. As pointed out in previous comments "and we can expand the Python test without a lot of fuss" is exactly the issue. Once we have two systems we end up with a tower babble. So until we can test with the current framework this patch is non-starter.

Cheers,
  -Brian

review: Disapprove

« Back to merge proposal