Code review comment for lp:~evarlast/charm-tools/warn-description-newlines

Revision history for this message
Tim Van Steenburgh (tvansteenburgh) wrote :

I appreciate this patch very much, but I don't think this is the correct way to solve this problem.

I think imposing arbitrary restrictions on which yaml contructs can be used will lead to a surprising user experience. This problem should be addressed by fixing the way the data is displayed by `juju get`, not by imposing restrictions on the data itself.

« Back to merge proposal