Make search the default subcommand

[dbgrandi]

With @AliSoftware adding the search subcommand we could make that the default and change the behavior of search to list all the plugins unless you pass in an optional regex to search with.

pod plugins would list out all the plugins

and

pod plugins neonichu would list any plugins that @neonichu authors.

Thoughts?

[AliSoftware]

Sounds great to me. Actually as I added the search command I thought about it and even thought about accepting an empty query, making it list all plugins in that case, so I'm all good for this.

[neonichu]

👍

[AliSoftware]

Still thinking that pod plugins neonichu shouldn't list plugins authored by neonichu unless we specify the --full flag to specifically tell to search in the 'author' field.

This would ensure consistency with pod search behavior, searching only in the plugin 's 'name' field if we don't use --full.

[dbgrandi]

Good point, and I agree with you. Bad example on my part. Let me rephrase that second one...

pod plugins stat would show any plugin with to stat in the name.

[fabiopelosin]

Wow @AliSoftware sweating the details!

[AliSoftware]

@irrationalfab always 😉

[dbgrandi]

Is it weird that a valid search this week (e.g. pod plugins popular) could end up doing something different (and possibly destructive) next week if we add new subcommands?

I'm trying to think of another unix command that follows this pattern (first positional arg can be a named subcommand if it exists or input to the default command) and I can't think of one off the top of my head.

@AliSoftware, you've got me overthinking the details now! 😄

[AliSoftware]

Agreed, I was thinking the same thing. Maybe setting search as the default subcommand whereas it can take arguments was not such a good idea after all.

This gives more and more credit to @irrationalfab 's suggestion of having a separate list subcommand (taking no argument, and being the new default subcommand) in addition to the search subcommand.

I vote for pod plugins list being the new default and rollbacking to pod plugins search QUERY having its parameter mandatory again. Thoughts?

@AliSoftware, you've got me overthinking the details now! 😄

hehehe

[neonichu]

@AliSoftware Sounds good.

[AliSoftware]

Done! (new commits in #13)