-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Default options for CLI commands (update/install) #130
Comments
I'm facing the same! I will add it! |
Thinking about it, maybe it's better to have an Environment variable to control this. What do tou think? |
My personal opinion would just be an argument to the script.
Env variable makes me have to keep separate settings on different files (in
this case zshrc needs to have a special line for a plug-in on tmux.
…On Thu, Jan 5, 2023, 08:12 Ferran Basora ***@***.***> wrote:
Thinking about it, maybe it's better to have an Environment variable to
control this. What do tou think?
—
Reply to this email directly, view it on GitHub
<#130 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD7WGR37GUNAK2UGRHQ3EXLWQZ667ANCNFSM6AAAAAATP5N3GE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
I thought about this a bit more and honestly anything is enough, even the env var because you can just easily prepend the command with it. |
I have to battle test a bit more this branch but you can check it out: #133 |
Is there way for me to test it locally?
I don't have much experience in OSS.
Might as well try to help out how I can since you are having all the work!
…On Thu, Jan 19, 2023, 21:58 Ferran Basora ***@***.***> wrote:
I have to battle test a bit more this branch but you can check it out:
#133 <#133>
—
Reply to this email directly, view it on GitHub
<#130 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AD7WGR7F5GPREBJ4F7CJ4KDWTG2H5ANCNFSM6AAAAAATP5N3GE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
You can checkout the source code and run the script passing the env variable described in the Readme. |
Looks ok to me ;) |
hello! just circling back on this to see if this will indeed make it to main or not. |
I automate most of my dotfiles including updating them and everytime I update tmux I have to see the tmux-thumbs screen and always press option 1 (compile).
It would be nice if this CLI command could accept a default option.
something like
or
This would make the update process silent and just compile without always having to ask if we want to update or not.
The text was updated successfully, but these errors were encountered: