Skip to content

Removed erroneous references to PREFIX environment variable#160909

Merged
Tyriar merged 1 commit intomicrosoft:mainfrom
CKeilbar:bugfix-terminal-prefix
Sep 26, 2022
Merged

Removed erroneous references to PREFIX environment variable#160909
Tyriar merged 1 commit intomicrosoft:mainfrom
CKeilbar:bugfix-terminal-prefix

Conversation

@CKeilbar
Copy link
Contributor

@CKeilbar CKeilbar commented Sep 14, 2022

Closes #160748.

This patch removes references to the PREFIX environment variable in the Bash and Zsh integrated terminals.
Without this change, the PS1 prompt for both of these shells contained $PREFIX, which is a common environment variable users tend to set.

To test these changes:

  1. Enable the integrated terminal for Bash or Zsh
  2. In the terminal, assign a value to a variable called PREFIX
  3. Nothing should happen to the prompt

@ghost
Copy link

ghost commented Sep 14, 2022

CLA assistant check
All CLA requirements met.

@Tyriar Tyriar added this to the September 2022 milestone Sep 15, 2022
@Tyriar Tyriar self-assigned this Sep 15, 2022
@Tyriar Tyriar requested a review from meganrogge September 26, 2022 14:05
@Tyriar Tyriar merged commit 1723a56 into microsoft:main Sep 26, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Nov 10, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PREFIX environment variable displayed in PS1 in integrated terminal

3 participants