Skip to content

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

Closed
cclauss wants to merge 5 commits intoopenaps:masterfrom
cclauss:modernize-Python-2-codes
Closed

Use print() function in both Python 2 and Python 3#22
cclauss wants to merge 5 commits intoopenaps:masterfrom
cclauss:modernize-Python-2-codes

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Jan 31, 2019

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

Legacy __print__ statements are syntax errors in Python 3 but __print()__ function works as expected in both Python 2 and Python 3.
@cclauss cclauss force-pushed the modernize-Python-2-codes branch from a9cc267 to c9d983e Compare January 31, 2019 12:20
@scottleibrand
Copy link
Collaborator

Do we need to do something else to enable Travis on this repo?

@cclauss
Copy link
Contributor Author

cclauss commented Feb 9, 2019

@openaps needs to visit https://travis-ci.org/openaps and flip the repo switch on or run the first test.

@scottleibrand
Copy link
Collaborator

Enabled Travis. Closing and reopening the PR to force a build.

@cclauss
Copy link
Contributor Author

cclauss commented Mar 10, 2019

@bewest
Copy link
Member

bewest commented Mar 11, 2019

Need to target dev branch as the standard workflow. See https://github.com/openaps/stdprojectexample and "git flow" for more details. Travis is enabled for this project.

@cclauss
Copy link
Contributor Author

cclauss commented Mar 11, 2019

The dev branch is 2 commits ahead, 3 commits behind master.

@cclauss
Copy link
Contributor Author

cclauss commented Mar 12, 2019

Closing in favor of #23 which is done on the dev branch, not the master branch.

@cclauss cclauss closed this Mar 12, 2019
@cclauss cclauss deleted the modernize-Python-2-codes branch March 12, 2019 21:58
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.

3 participants