Skip to content
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

Add Attach remote option #65

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Add Attach remote option #65

wants to merge 5 commits into from

Conversation

cousine
Copy link

@cousine cousine commented Jan 15, 2024

Add "Attach remote" as a default option as well as an input to enter the connection address

@leoluz
Copy link
Owner

leoluz commented Mar 25, 2024

@cousine Hi! Not sure I understand what is the advantage of having the dlv uri and the remote path defined in runtime instead of just having a dedicated user configuration with the values provided declaratively. Would you please elaborate why this is a good feature to have?

@cousine
Copy link
Author

cousine commented Mar 25, 2024

@leoluz the main use case I had is that we have several executables in the same project, and it was more convenient to be able to on-the fly connect to a remote session especially that I use the remote session occasionally for debugging within a container

@leoluz
Copy link
Owner

leoluz commented Jul 19, 2024

@cousine There is another on-going PR (#78) that I think will address your use-case.
Can you please take a look and confirm?
tks!

@cousine
Copy link
Author

cousine commented Sep 13, 2024

@leoluz thanks for sharing! looks like it does solve the issue, but the caveat here is the introduction of a new config format (vscode launch), tbh thats not ideal as it means I litter my repo with several config files just to configure my nvim env.

Not sure if anyone else would have this exact nit.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants