-
Notifications
You must be signed in to change notification settings - Fork 8
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
almostontop automatically substitutes variables #5
Comments
Thanks for the issue. I need some time to figure it out, whether it a feature or a bug. 😂 |
Sure, thanks. To add a bit more context to help you with your decision: It's fine when almostontop replaces commands like $ sudo !! with $ sudo #previous_command (this is the happy case, maybe even a feature if you will) But the problem is when it replaces commands like $ #some_private_util mkdir #some_path%ttl=1d with $ #some_private_util mkdir #some_path[#current_time]tl=1d In this command "%" after the path is treated as a delimiter followed by args (like ttl = 1d in this example) and almostontop breaks that by replacing %t with #current_time -- this, I would call a bug -- at least without a way to suppress this behavior and keep almostontop on. |
@avamsi , should be fixed now. Please check out the new version. |
could be easily checked with |
That was pretty quick! Thanks @Valiev! |
For example, $ echo $EDITOR without almostontop just gets executed as $ echo $EDITOR
But with almostontop, it gets replaced with $ echo ~/bin/micro
The text was updated successfully, but these errors were encountered: