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

bash shellIntegration: Prevent error messages when running with set -u #233527

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AndrewDDavis
Copy link

This change causes bash to substitute a null string for variables that are referenced, but unset. This prevents error messages when running bash with set -u.

e.g. "bash: VSCODE_INJECTION: unbound variable"

To test:

  • place set -u in your ~/.bashrc, to show an error message when unset variables are referenced
  • open a new terminal in VS-Code, with shell-integration enabled

e.g. "bash: VSCODE_INJECTION: unbound variable"
@AndrewDDavis AndrewDDavis changed the title Prevent error messages when running with set -u bash shellIntegration: Prevent error messages when running with set -u Nov 10, 2024
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