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

say command during install should be removed or made configurable by flag #230

Open
guerreit opened this issue Feb 23, 2023 · 9 comments
Open

Comments

@guerreit
Copy link

  • Text to speech in install hook should respect OS settings
  • Text to speech in install hook should be configurable by flag

say "$(tput setaf 2)✓$(tput sgr 0) Installed ${hook} hook to ${dest}"

@micxer
Copy link

micxer commented Apr 14, 2023

Why is this in there anyway? Doesn't make much sense. An echo in that place makes way more sense.

@kallenboone
Copy link

kallenboone commented Apr 18, 2023

At least make it fun with -v Trinoids or -v Bahh.

But more seriously, please make it optional or remove it.

@Hacksore
Copy link

Trying to get this merged. That's not a guarantee but just wanted to share that with those who are subscribed to this issue.

@justinblough
Copy link

My entire team is plagued by this and absolutely irritated as it is part of our install process for packages.

@sorenisanerd
Copy link

I have to assume that say doesn't to text-to-speech on whichever platform was used for development for this, otherwise why add terminal control characters that won't be echoed, but will be read aloud?

It's kinda weird, honestly.

I recommend adding a directory with a say executable in it that just calls echo, and then add that to your path before calling git secrets --install.

@Rylon
Copy link

Rylon commented Jan 4, 2024

I think this got fixed in this commit: 65891e2 however I don't see a new release tag since 1.3.0 which is dated Feb 10th 2019.

Would it be possible for someone to cut a new release and update the Homebrew formulae for it please? https://github.com/Homebrew/homebrew-core/blob/9f15fa987cbed2d11971a365d33b140a3eba3c74/Formula/g/git-secrets.rb

@mtdowling I'm pinging you as the author listed in the README if you are able to help at all?

Thanks very much :)

@alvincrespo
Copy link

Still an issue. Here is to hoping we get a new tag soon 🤞

@coringham
Copy link

coringham commented Oct 17, 2024

For anyone else poking around and having been waiting since this ticket opened:
homebrew workaround found from one of the devs I work with

brew unlink git-secrets
brew install --HEAD git-secrets

I finally have silence again.

@kallenboone
Copy link

For anyone else poking around and having been waiting since this ticket opened: homebrew workaround found from one of the devs I work with

brew unlink git-secrets
brew install --HEAD git-secrets

I finally have silence again.

It brings me joy to see a resolution to this.

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

No branches or pull requests

9 participants