search

nextstrain / fauna

RethinkDB database to support real-time virus analysis
GNU Affero General Public License v3.0
33 stars 13 forks source link

Use print() function in both Python 2 and Python 3 #102

Closed cclauss closed 3 years ago

cclauss commented 4 years ago

Legacy print statements are syntax errors in Python 3 but print() function works as expected in both Python 2 and Python 3.

Description of proposed changes

What is the goal of this pull request? What does this pull request change?

Related issue(s)

Fixes #
Related to #

Testing

What steps should be taken to test the changes you've proposed?
If you added or changed behavior in the codebase, did you update the tests, or do you need help with this?

Thank you for contributing to Nextstrain!

jstoja commented 4 years ago

Looks good to me 👍

ghost commented 4 years ago

@trvrb should this PR be approved or are we waiting some major migration to python 3?

cclauss commented 4 years ago

Python 2 died 137 days ago.

trvrb commented 3 years ago

Thanks for the fixes here! 🙏🏻 I've rebased these changes into master with 0bd4375414e40478383e3c2f6b1d1f54d3a7f26a.