Juju Lint

Code review comment for ~mertkirpici/juju-lint:lp/1987951

  1. Git
  2. lp:~mertkirpici/juju-lint
  3. lp/1987951
  4. Merge into master
Revision history for this message
Gabriel Cocenza (gabrielcocenza) wrote :

The patch LGTM, but unit tests are failing because of the changes and because now we need 100% line coverage.

Regarding the bugs, please fill the first one. The second one after talking with Martin, it's not that bad because it's the public certificate of the controller.

Thanks!

review: Needs Fixing

« Back to merge proposal