-
Notifications
You must be signed in to change notification settings - Fork 1
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
doubt about the approach taken #1
Comments
Ah sorry, I forgot to clarify how this solution came to be. It started from this discussion. And also afaik connecting to Helix from external programs is not yet supported which have feature such as switching file, this script use macro to do that. This just solved my issue generally enough that I just stick with it. I'd love to update to a better solution! I've read your script briefly and it seems like your script only contains the launching part which my script actually doesn't have. Here my script aim to let user create their own launching script to let users use whatever terminal they want which of course have different configuration and connect that to my script with the flag If Helix still doesn't support connection then I might update the script with copy-paste to fix the waiting issue. I'll update the README to clarify what this thing solves. |
Yeah, makes sense @Multirious! |
Hey @Multirious I just wanted to ask about why you approached this using macros and automation
I had a similar issue, I wanted to use helix as the external editor for Godot, so I found your repo but decided to give it a try with a different approach
my take
plus the following config on Godot
I just spawn a alacritty.exe + hx.exe command with the arguments Godot provides when calling an external editor and it seems to work a-okay without having to wait for the automation/macro to run (though the repo is not really meant to be used by others as I haven't made it general, just wanted to quickly fix it for my issue)
But I got curious so I wanted to ask, if you don't mind, why'd you automate it instead?
The text was updated successfully, but these errors were encountered: