NeonXP/nvim-dap-go
1
0
Fork 0
Code Issues 5 Pull requests 5 Projects Releases Packages Wiki Activity Actions

Allow users to pass .env files to debug config #115

Open
theoribeiro wants to merge 10 commits from theoribeiro/main into main
pull from: theoribeiro/main
merge into: NeonXP:main
Conversation 1 Commits 10 Files changed 2 +150 -16
theoribeiro commented 2025-03-06 19:30:49 +03:00 (Migrated from github.com)
Copy link

This PR adds a feature that has been very useful to me and is available in a very similar way in VSCode.

Description

With the Go plugin in VSCode you can pass an argument called envFile to pass a .env file to delve. The Delve adapter In reality doesn't have this feature, so the extension is reading the .env file and setting up variables to pass as the env argument, which Delve supports.

This PR emulates the same behavior by reading the envFile argument and enriching the env config parameter before passing it on to nvim-dap.

This PR adds a feature that has been very useful to me and is available in a very similar way in VSCode. ## Description With the Go plugin in VSCode you can pass an argument called `envFile` to pass a .env file to delve. The Delve adapter In reality doesn't have this feature, so the extension is reading the .env file and setting up variables to pass as the `env` argument, which Delve supports. This PR emulates the same behavior by reading the `envFile` argument and enriching the `env` config parameter before passing it on to nvim-dap.
👍 3 ❤️ 1 🚀 2
dennypenta commented 2025-08-09 23:38:25 +03:00 (Migrated from github.com)
Copy link

@leoluz does anything block this PR? might remove lots of code from config that everyone adds nowadays to support env files

@leoluz does anything block this PR? might remove lots of code from config that everyone adds nowadays to support env files
👍 1
Merge branch 'main' into theoribeiro/main
Some checks failed
Lint / Stylua (pull_request) Has been cancelled
Details
Lint / Super Linter (pull_request) Has been cancelled
Details
19d9419c4c
Some checks failed
Lint / Stylua (pull_request) Has been cancelled
Details
Lint / Super Linter (pull_request) Has been cancelled
Details
This pull request has changes conflicting with the target branch.
  • lua/dap-go.lua
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin theoribeiro/main:theoribeiro/main
git switch theoribeiro/main

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff theoribeiro/main
git switch theoribeiro/main
git rebase main
git switch main
git merge --ff-only theoribeiro/main
git switch theoribeiro/main
git rebase main
git switch main
git merge --no-ff theoribeiro/main
git switch main
git merge --squash theoribeiro/main
git switch main
git merge --ff-only theoribeiro/main
git switch main
git merge theoribeiro/main
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

Something isn't working

  
documentation

Improvements or additions to documentation

  
duplicate

This issue or pull request already exists

  
enhancement

New feature or request

  
good first issue

Good for newcomers

  
help wanted

Extra attention is needed

  
invalid

This doesn't seem right

  
question

Further information is requested

  
wontfix

This will not be worked on

No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
NeonXP/nvim-dap-go!115
No description provided.