-
Notifications
You must be signed in to change notification settings - Fork 41
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
tenv failing with pre-commit #328
Comments
A full log of it
|
Here's the failing part:
|
Also having an issue with this.
|
for the record: happens on Linux x64 as well (original issue was on MacOS) Edit:
The problematic change seems to be this: f665b6e#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R644 |
Other tools are throwing too with similar errors. I've had problems running a VSCode extension that is using the terraform binary and today I've found out I can no longer run Terraform acceptance tests: Overriding the binary with |
This looks connected with #255 and #304 - Seems to be that the default was changed as @daniel-ciaglia mentioned with a workaround for CI pipelines, but it seems to be affecting standard console use as well. |
I can confirm that there are problems with VSCode as well. |
This also affects the Terraform plugin for Vim's format-on-save, and it seems simply redirecting stdin in a terminal as well:
|
This issue seems to be related to the fact that tenv doesn't just |
Can you test v4.2.0-beta2 ? |
Works for me :)
|
@dvaumoron thank you, v4.2.0-beta2 works the Vim plugin and the redirection test in my previous comment. |
Describe the bug
tenv fails locally with pre-commit, when
export TENV_DETACHED_PROXY=false
is not set.To Reproduce
git init
main.tf
:git add -A && git commit -m "test"
pre-commit run --all-files
(it will fail)export TENV_DETACHED_PROXY=false
pre-commit run --all-files
(it won't fail)Expected behavior
tenv shouldn't fail with pre-commit, without
export TENV_DETACHED_PROXY=false
Environment (please complete the following information):
The text was updated successfully, but these errors were encountered: