Skip to content

Commit c87e8f7

Browse files
authored
fix(vscode): Keep HOME variable when starting VSCode (#2694)
## Summary Removing the HOME variable has adverse effects on many tools, most notable git. Without it, tools can't find the configurations. ## How was it tested? - Overrode the derivation and applied the patch - Restarted VSCode - Triggered the "Reopen in devbox shell" action - `echo $HOME` in the shell - Make this commit with the opened VSCode ## Community Contribution License All community contributions in this pull request are licensed to the project maintainers under the terms of the [Apache 2 License](https://www.apache.org/licenses/LICENSE-2.0). By creating this pull request, I represent that I have the right to license the contributions to the project maintainers under the Apache 2 License as stated in the [Community Contribution License](https://github.com/jetify-com/opensource/blob/main/CONTRIBUTING.md#community-contribution-license).
1 parent 030acae commit c87e8f7

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

internal/boxcli/integrate.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,7 @@ func runIntegrateVSCodeCmd(cmd *cobra.Command, flags integrateCmdFlags) error {
108108
// PATH after VSCode opens and resets it to global shellenv. This causes the VSCode
109109
// terminal to not be able to find devbox packages after the reopen in devbox
110110
// environment action is called.
111-
return ok && (strings.HasPrefix(k, "DEVBOX_OG_PATH") || k == "HOME" || k == "NODE_CHANNEL_FD")
111+
return ok && (strings.HasPrefix(k, "DEVBOX_OG_PATH") || k == "NODE_CHANNEL_FD")
112112
})
113113

114114
// Send message to parent process to terminate

0 commit comments

Comments
 (0)