Code review comment for ~tiago.pasqualini/layer-snap:snap_list

Revision history for this message
Tiago Pasqualini da Silva (tiago.pasqualini) wrote :

Hi Tom,

Fixed the docstring and rebased on top of master.

Regarding the exception, the code already checks if the snap is installed and returns nothing in that case, so it is not expected to fail if used correctly.

Please let me know if changed are enough.

Thanks,
Tiago

« Back to merge proposal