Merge into trunk : charm-search : Code : juju-jitsu

Merge lp://staging/~hazmat/juju-jitsu/charm-search into lp://staging/juju-jitsu

Proposed by Kapil Thangavelu
Status: Merged
Approved by: Juan L. Negron
Approved revision: no longer in the source branch.
Merge reported by: Mark Mims
Merged at revision: not available
Proposed branch: lp://staging/~hazmat/juju-jitsu/charm-search
Merge into: lp://staging/juju-jitsu
To merge this branch: bzr merge lp://staging/~hazmat/juju-jitsu/charm-search
Reviewer Review Type Date Requested Status
Juan L. Negron (community) Approve
Review via email: mp+117741@code.staging.launchpad.net

Description of the change

Simple charm search command.

To post a comment you must log in.
Revision history for this message
Kapil Thangavelu (hazmat) wrote :

Please take a look.

Revision history for this message
Juan L. Negron (negronjl) wrote :

Reviewing this now.

-Juan

Revision history for this message
Juan L. Negron (negronjl) wrote :

Hi Kapil:

This is pretty cool:

Per our conversation on IRC, here are my suggestions:
- --format is not working ... this is not a blocker really as the results are one per line and it's easy enough to parse.
- --help with some text detailing the search options would be nice. I didn't know about owner, series, etc.

Thanks,

Juan

review: Needs Fixing
Revision history for this message
Kapil Thangavelu (hazmat) wrote :

Thanks added documentation and format options.

Revision history for this message
Juan L. Negron (negronjl) wrote :

Nice work.
Thanks Kapil.

Approved.

-Juan

review: Approve
Revision history for this message
Kapil Thangavelu (hazmat) wrote :

*** Submitted:

Simple charm search command.

Subscribers

People subscribed via source and target branches