Skip to content

Comments

Mention Ghostty#172

Merged
rcaloras merged 1 commit intorcaloras:masterfrom
jparise:readme-ghostty
Dec 7, 2025
Merged

Mention Ghostty#172
rcaloras merged 1 commit intorcaloras:masterfrom
jparise:readme-ghostty

Conversation

@jparise
Copy link
Contributor

@jparise jparise commented Aug 3, 2025

Also, remove Fig, which was shut down in September 2024.

Also, remove Fig, which was shut down in September 2024.
Copy link
Contributor

@akinomyoga akinomyoga left a comment

Choose a reason for hiding this comment

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

I didn't know Fig had been shut down!

Since you are a Ghostty contributor, it feels a bit unfair to add only Ghostty in updating the list of projects. Today, many projects are relying on bash-preexec for their Bash integration. As far as I know, Warp also includes bash-preexec. Starship, VS Code Terminal, and Atuin also rely on bash-preexec as an option. Of course, it is not a complete list. Could you make an effort to search for projects that use bash-preexec and add them to the list? edit: Also, Bash-it and Oh-My-Bash use bash-preexec.

@ghthor
Copy link

ghthor commented Sep 20, 2025

Since you are a Ghostty contributor, it feels a bit unfair to add only Ghostty in updating the list of projects. Today, many projects are relying on bash-preexec for their Bash integration. As far as I know, Warp also includes bash-preexec. Starship, VS Code Terminal, and Atuin also rely on bash-preexec as an option. Of course, it is not a complete list. Could you make an effort to search for projects that use bash-preexec and add them to the list? edit: Also, Bash-it and Oh-My-Bash use bash-preexec.

I mean, I get what you're saying, but at the same time I don't get why this PR isn't just merged. None of those other projects that are using bash-preexec have taken the time to come update this documentation.

@rcaloras rcaloras merged commit aad114d into rcaloras:master Dec 7, 2025
1 check passed
@rcaloras
Copy link
Owner

rcaloras commented Dec 7, 2025

Happy to add this. We can also update the Readme to have a 'Used By' section to acknowledge all the dependent projects and who's leveraging bash-preexec.

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.

4 participants