berks shelf show should take an optional VERSION argument

[sethvargo]

The berks shelf show command should take an optional VERSION argument

• If not provided, information about the latest version of the cookbook will be shown
• If it is provided, information about that version of the cookbook will be shown

[sethvargo]

@reset this is already done, but I changed this into a PR to fix a small bug when the passed version does not exist and expanded the test cases.

[ivey]

This is an incorrect statement. Shouldn't it be "Version 1.2.3 of fake is not in the shelf"?

[ivey]

Are you going to drop the -v aliases on show and delete here, or in a separate PR?

[reset]

@ivey @sethvargo doesn't matter where it happens, but -v should be removed as an alias on any of our commands since it's also used for --version

[sethvargo]

@reset @ivey -v and --version are both already used. In this case, thor "does the right thing" and chooses the method option over the map. I think this is the preferred behavior, as that's how gem works too:

$ gem -v
1.8.23

$ gem install berkshelf -v 1.4.4
Fetching: yajl-ruby-1.1.0.gem (100%)
Building native extensions.  This could take a while...
Fetching: berkshelf-1.4.4.gem (100%)
Successfully installed yajl-ruby-1.1.0
Successfully installed berkshelf-1.4.4
2 gems installed

[reset]

As long is there's no conflict I'm ok with it. @ivey?

[sethvargo]

@reset @ivey fixed those issues with the output being inconsistent.

[ivey]

I'm fine with it if it works.