100 shell autocomplete, type fixes, Pillow deprecation cleanup #101
Inorishio
No Assignees
AramJonghu
AramJonghu
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: zach/z-bar-qt#101
Reference in New Issue
Block a user
Delete Branch "100-cli-autocompletion"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Changes
A few changes. Would also like some preferences as some of the current changes could be seen as a forced change.
zshell-clicommands in general. Check here for an example:This would apply to all non flag commands. A flag will give autocomplete suggestions once the user inputs a
-. Example here:Preference question
In this current code, the first zshell-cli command auto installs autocompletion and writes (into) two files. While convenient to me, this could be frowned upon that it writes into files without intentional user input. An alternative could be a hint message or a first time zshell-cli prompt to install autocomplete (only once). If this is preferred, I will adjust.
Closes #100
Personally think we should have it as a command like
zshell-cli install-completionor similar, and then inform users about it in the project's readme. This is very useful and I like the changes, but I also want the CLI tool to be as unobtrusive as possible. It's really only meant to be used manually by people working on the shell or tool itself.Ill look into it and make the changes.
@zach since you did not leave a "review", @ing you instead. Changes are made. I did add a tip if autocomplete is not installed. Should not be of any harm.