Skip to content

Deno.isTTY() possibly reports incorrect value when running in GitHub Actions #2885

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

Closed
lucacasonato opened this issue Sep 7, 2019 · 4 comments

Comments

@lucacasonato
Copy link
Member

This is a blocker for #2863.

In the is_tty_test.TestIsTty the //tests/is_tty.ts file gets run. This file logs the value of Deno.isTTY().stdin. Deno reports true in the Github Actions environment. The same check happens in python by calling stdin.isatty(). This reports false in the Github Actions environment. As these values do not equal the test fails.

Either python or deno are reporting an incorrect value here. What are the qualifications to return true in Deno.isTTY()? I am working on the Github Actions script in this repo: https://github.com/lucacasonato/deno/tree/github-ci.

@ry
Copy link
Member

ry commented Sep 9, 2019

@lucacasonato I suggest just commenting out the tests for now... It may be that github actions actually is a tty.

@bartlomieju
Copy link
Member

@lucacasonato still relevant?

@lucacasonato
Copy link
Member Author

@bartlomieju Not sure. I'll try test it today.

@lucacasonato
Copy link
Member Author

@bartlomieju Just looked into it. The test seems to have been removed all together so this is not relevant anymore.

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

3 participants