-
Notifications
You must be signed in to change notification settings - Fork 510
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
Improve support for Alfred #348
Comments
Hi, thanks for navi. I had a suggestion here, not sure sure if it's supported yet. |
you can add a output |
@lslz627 Thanks, I can modify that. Checking if upstream workflow will support it. It would be great if I don't have to maintain this, other folks can use it as well. |
@perfectayush did you manage to make it work? If so, would you mind opening a PR? I can then maintain it 👍 |
@denisidoro the solution recommended by @lslz627 works. But it will break the existing terminal command workflow. I am not sure what would be the best way to do this in final terminal/else switch. |
Unfortunately I don't have any metrics regarding navi's usage but I bet that, relatively speaking, few people use this Alfred workflow (because it's experimental and hacky). I'm OK with breaking changes if they are for the greater good. |
@denisidoro I personally feel that best course would be to copy the command to clipboard and paste it in front most app. This is very useful for pasting the command in terminal or a text editor. This also allows for inspecting command before running it. Running it in terminal once evaluated or eval "$(snippet)" is probably not a great idea without user inspection. From usage point of view, I have also added a hotkey in the beginning to initiate this workflow. I prefer I have exported the workflow, I will add a PR if this is acceptable. |
Unfortunately, I'm dropping support for Alfred in #509. Reasons:
Any additional info related to Alfred will be documented in this page. I'm sorry for all navi+Alfred users out there! |
.bashrc
--map
and--column
The text was updated successfully, but these errors were encountered: