Push with hooks warning: fail to read /dev/stdin

[QuentiumYT]

The problem

Hello,
I am testing every beta and this warning appears since beta3 or beta4 and still present in beta5. It didn't show before with my repository configured with hooks for several month now.

I am using husky in that NodeJS project, the content of pre-prush is:

#!/usr/bin/env sh
. "$(dirname "$0")/h"

Thanks

Release version

Appeared in 3.5.5-beta3 or beta4

Operating system and distro

Linux Mint 22.3

Steps to reproduce the behavior

No response

Specific to GitHub Desktop Plus?

Yes

Log files

No response

Screenshots

No response

Additional context

No response

[pol-rivero]

I think this might be related to the new Hooks support added by the upstream project. Can you try disabling "Load Git hook environment variables from shell" in the Git Hooks settings?

[QuentiumYT]

Thanks for the reply, I didn't think it was part of GitHub Desktop. Will try that tomorrow and give you an answer about it.
Have a nice evening

[pol-rivero]

Okay!
Have a nice evening too.

[QuentiumYT]

This indeed resolve my issue, the hooks running under the commit box no longer show now but that's normal.

I just had this issue when pushing to upstream, maybe I have to create an issue in GitHub Desktop as it's a feature implemented there?

[pol-rivero]

If you have access to a Windows or macOS machine, you could try to reproduce the issue in the official GitHub Desktop app.
However, I suspect this will be a Linux-only problem, so the upstream maintainers won't be interested in fixing it.

Let's leave this issue open for now. This is currently a low-priority problem, since it can be easily fixed by just disabling the hooks integration. However, it would be nice to make this work by default in the future.

[QuentiumYT]

Oh yes you're right, might have to be fixed here then :p
No problem, thank you very much for that. Don't hesitate to ask me to test new builds or more info :)

[pol-rivero]

This is now fixed in v3.5.7