zimfarm icon indicating copy to clipboard operation
zimfarm copied to clipboard

Zimfarm worker cmd line autocompletion

Open kelson42 opened this issue 5 years ago • 10 comments

If I do a zimfarm logs <TAB> <TAB> it would be nice to get autocompletion for the aliases available (manager, maybe others, see above) and the container IDs available.

kelson42 avatar Feb 20 '20 13:02 kelson42

Autocompletion is done via shell configuration and to be able to do that we would need to make a zimfarm client pip package (at least).

kelson42 avatar Feb 20 '20 13:02 kelson42

:1 on this issue.

A pip package is not mandatory for that but might make more sense that our single script.

Just so we're clear on this, user will need to install the completion file outside of pip for it to work with his shell.

rgaudin avatar Feb 20 '20 14:02 rgaudin

Just so we're clear on this, user will need to install the completion file outside of pip for it to work with his shell.

This project is imho for experience users, installing a script by hand is acceptable to me.

JensKorte avatar Feb 20 '20 15:02 JensKorte

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

stale[bot] avatar Apr 22 '20 19:04 stale[bot]

Just so we're clear on this, user will need to install the completion file outside of pip for it to work with his shell.

Really! This is quite a deception.

kelson42 avatar Dec 30 '20 13:12 kelson42

Why the question label?

rgaudin avatar Dec 30 '20 13:12 rgaudin

@rgaudin I really wonder if we should implement it, if its setup is so cumbersome.

kelson42 avatar Dec 30 '20 13:12 kelson42

Currently, setup is copy-pasting a small set of commands. We could still have that and keep a shell install.

You mentioned a pip package, I believe, thinking it would be simpler to install. It would (although it would require python and pip which we don't currently use) but pip is not able to install files outside the python installation paths. Our code could do that upon launch though.

rgaudin avatar Dec 30 '20 13:12 rgaudin

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

stale[bot] avatar Mar 19 '21 23:03 stale[bot]

This issue has been automatically marked as stale because it has not had recent activity. It will be now be reviewed manually. Thank you for your contributions.

stale[bot] avatar Jun 12 '22 20:06 stale[bot]