Fix dev selector typo

[teresaromero]

Fixes #2778

Proposed changes

• Separate selection logic from GetDevFromManifest. This function will though an error (ErrNoDevSelected) if devName is empty and manifest has multiple devs defined.
• new SelectDevFromManifest fallback function when err is ErrNoDevSelected, where label is a param that represents the question of the selector

[codecov]

Codecov Report

Merging #3033 (fed84ba) into master (fcc16ba) will decrease coverage by 0.03%.
The diff coverage is 32.65%.

@@            Coverage Diff             @@
##           master    #3033      +/-   ##
==========================================
- Coverage   32.79%   32.75%   -0.04%     
==========================================
  Files         188      188              
  Lines       19762    19788      +26     
==========================================
+ Hits         6480     6481       +1     
- Misses      12517    12543      +26     
+ Partials      765      764       -1     

Impacted Files	Coverage Δ
cmd/doctor.go	0.00% <0.00%> (ø)
cmd/down.go	0.00% <0.00%> (ø)
cmd/push.go	0.00% <0.00%> (ø)
cmd/restart.go	0.00% <0.00%> (ø)
cmd/status.go	0.00% <0.00%> (ø)
cmd/up/up.go	10.62% <0.00%> (-0.09%)	⬇️
cmd/utils/dev.go	33.03% <83.33%> (+0.60%)	⬆️
cmd/exec.go	14.03% <85.71%> (-0.88%)	⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[ifbyol]

@rlamana @RinkiyaKeDad Can you review the text for the elector on each situation, please?

[ifbyol]

Can you include a comment in this exported function, please?

[RinkiyaKeDad]

I've added suggestions for the messages.

Another thing I was thinking was that we could just change all the messages to "Select a development container:" - since the user running the command would know from the command itself what exactly they are selecting the container for. What do you all think?

[teresaromero]

I've added suggestions for the messages.

Another thing I was thinking was that we could just change all the messages to "Select a development container:" - since the user running the command would know from the command itself what exactly they are selecting the container for. What do you all think?

@RinkiyaKeDad Thnks for the suggestions! I've fixed the strings accordingly.

Regarding the message, i think adding the action that is being performed, although the user knows from the command, is more descriptive.