Use completing-read in interactive prompt for ruby version

[til]

One more 😄 .

This way one does not have to guess or remember which rubies are available. Works for me with helm, and also without helm.

[plexus]

I don't think you need cl-mapcar here, regular mapcar should do.

From the manual

Given a single seq argument it is equivalent to mapcar; given n sequences, it calls the function with the first elements of each of the sequences as the n arguments to yield the first element of the result list, then with the second elements, and so on.

[til]

I saw that cl-mapcar is used here as well although mapcar is sufficent (I think): https://github.com/plexus/chruby.el/blob/master/chruby.el#L78-L81 and followed. What's the general recommendation - use cl-* functions only when needed? Anyway I'll change it in the PR, hold on.

[plexus]

Thanks for the PR! Definitely an improvement. I now remember why these functions weren't documented. They were contributed by someone else, and I guess that person didn't update the docs. I had forgotten they even existed :)

Yeah, in general I'd say only use the cl functions when there's no regular alternative. But it doesn't matter so much to be honest. I just added cl-lib as an explicit dependency now, that wasn't the case yet.

[til]

Thanks for the feedback! Pushed a change to use mapcar.

[plexus]

yay!