search

pharo-project / pharo-launcher

Lets you manage your pharo images and download new ones
https://pharo-project.github.io/pharo-launcher/
MIT License
108 stars 46 forks source link

[cmd-line] Select image for (more) commands #676

Open PalumboN opened 4 months ago

PalumboN commented 4 months ago

Describe the request

Now when you execute some image related command, the prompt ask you for the image index if you don't pass it as parameter.

➜  ~ pl image launch
1  p11-druid-with-mutalk               
2  p11-druid-specialize-primitiveAt    
...
Select the image number:

This feature works with: launch, delete and recreate.

However, it's missing for the commands: info, copy and package (I think that I don't forget anyone).

Would be nice to have the same behaviour in all of them.

Version information:

➜ pl -v
3.1.1-476-g723df186
demarey commented 4 months ago

I just added it for the info command. For the copy and package command, it is more difficult because the image name is declared as a mandatory positional in the command spec. Also, it will mean that copy command could be used like: copy newImageNameand the select the existing image instead of copy existingImageName newImageName. I'm not sure it would be best. Maybe we should wait and see how PL CLI users want to use this command?

PalumboN commented 4 months ago

Niceee!!! 🙌