Skip to content

respect fish_prompt, re-enable automatic fish shell integration#185355

Merged
meganrogge merged 12 commits intomainfrom
merogge/fish-s
Jun 20, 2023
Merged

respect fish_prompt, re-enable automatic fish shell integration#185355
meganrogge merged 12 commits intomainfrom
merogge/fish-s

Conversation

@meganrogge
Copy link
Collaborator

fixes #139400

@meganrogge meganrogge self-assigned this Jun 16, 2023
@meganrogge meganrogge requested a review from Tyriar June 16, 2023 16:46
@meganrogge meganrogge added this to the June 2023 milestone Jun 16, 2023
Tyriar
Tyriar previously requested changes Jun 16, 2023
@Tyriar
Copy link
Member

Tyriar commented Jun 16, 2023

@mkhl @zgracem a review would be appreciated if you can spare the time

Copy link
Contributor

@mkhl mkhl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Super tiny nits but LGTM!

(I don't love duplicating the default fish prompt but can't see a way around it)

@meganrogge meganrogge enabled auto-merge June 19, 2023 21:22
@meganrogge meganrogge merged commit bd86e0f into main Jun 20, 2023
@meganrogge meganrogge deleted the merogge/fish-s branch June 20, 2023 16:39
@github-actions github-actions bot locked and limited conversation to collaborators Aug 4, 2023
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.

Support fish shell integration automatic injection

4 participants