support sys.stdout.isatty() #351

Closed
reidpr opened this Issue Mar 29, 2013 · 1 comment

Projects

None yet

2 participants

@reidpr
reidpr commented Mar 29, 2013

Steps to reproduce:

  • Call sys.stdout.isatty() from within a Disco job.

Actual behavior:

AttributeError: 'MessageWriter' object has no attribute 'isatty'

Expected behavior:

  • Return value is 0.

Comments:

  • This is a common way to determine if various terminal stuff on standard out is supported. In a Disco job, the answer is no, so isatty() should exist on MessageWriter (or something -- I'm still a n00b with Disco) and return 0.
Owner
pooya commented Jun 6, 2014

commit a150a37 adds this feature. Thanks.

@pooya pooya closed this Jun 6, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment