Skip to content

Add isatty() method to PythonQtStdIn class #50

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

Merged
merged 1 commit into from
Jun 24, 2021

Conversation

lassoan
Copy link
Contributor

@lassoan lassoan commented Jun 23, 2021

pip and some other Python modules use this attribute to check if they are running in an interactive console.

Co-authored-by: Fernando Perez-Garcia fernando.perezgarcia.17@ucl.ac.uk

fixes #48

@jcfr
Copy link

jcfr commented Jun 23, 2021

Thanks @lassoan

Could you update the commit to include the following:

Co-authored-by: Fernando Perez-Garcia <fernando.perezgarcia.17@ucl.ac.uk>

For reference, here is a similar commit we contributed back almost 10 yrs ago ... commontk/PythonQt@7132dba

@lassoan
Copy link
Contributor Author

lassoan commented Jun 23, 2021

Could you update the commit to include the following:

This was already in the commit comment.

@jcfr
Copy link

jcfr commented Jun 23, 2021

This was already in the commit comment.

Issue is that the Co-authored-by: is a "commit trailer" and should be at the end of the commit message otherwise it is ignored.

Add isatty() method to PythonQtStdIn class 

pip and some other Python modules use this attribute to check if they are running in an interactive console.
+
+ fixes #48
+
Co-authored-by: Fernando Perez-Garcia <fernando.perezgarcia.17@ucl.ac.uk>
-
- fixes #48

pip and some other Python modules use this attribute to check if they are running in an interactive console.

fixes MeVisLab#48

Co-authored-by: Fernando Perez-Garcia <fernando.perezgarcia.17@ucl.ac.uk>
@lassoan lassoan force-pushed the 48-add-isatty-to-stdin-upstream branch from 3b58d29 to bc65dba Compare June 24, 2021 01:47
@lassoan
Copy link
Contributor Author

lassoan commented Jun 24, 2021

I did not know that the order matter. I've moved the coauthor message to the end.

@usiems usiems merged commit 7012000 into MeVisLab:master Jun 24, 2021
@lassoan lassoan deleted the 48-add-isatty-to-stdin-upstream branch June 24, 2021 15:11
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

Successfully merging this pull request may close these issues.

Add .isatty() to PythonQtStdIn
3 participants