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

Revision history for this message
Mert Kirpici (mertkirpici) wrote :

Hi all,
I tried to address everything in the update(s), which I will push in a sec.

Also with Gabriel's suggestion to implement this feature as a comma separated list rather than specifying the "-c" command line option multiple times; i.e.

  $ juju-lint -c "file1.yaml,https://file2.yaml" ...

I think we can avoid code change in the charm-juju-lint completely. Just a description change in the config.yaml suggestion should be enough.

« Back to merge proposal