Skip to content

Conversation

@babakks
Copy link
Contributor

@babakks babakks commented Jul 23, 2022

Partially fixes #155530

Copy link
Member

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

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

This looks great! I'm off for a week but will check it out then 👍

@Tyriar Tyriar added this to the August 2022 milestone Jul 24, 2022
@babakks
Copy link
Contributor Author

babakks commented Jul 31, 2022

@Tyriar I updated the PR by extracting the sanitizeFishHistoryCmd function and adding some unit tests to verify its RegExp substitution works correctly.

Copy link
Member

@Tyriar Tyriar left a comment

Choose a reason for hiding this comment

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

Thanks @babakks, awesome PR. Just pushed a shell type detection edge case fix I hit.

@Tyriar Tyriar merged commit 9ad7a04 into microsoft:main Aug 5, 2022
@babakks
Copy link
Contributor Author

babakks commented Aug 5, 2022

@Tyriar Happy to hear that. You're welcome.

@babakks babakks deleted the add-fish-history branch August 5, 2022 08:02
joyceerhl pushed a commit that referenced this pull request Aug 10, 2022
🎁 Add support for `fish` shell history
@github-actions github-actions bot locked and limited conversation to collaborators Sep 19, 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.

Support other shell types for terminal shell integration history

3 participants