Skip to content

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

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

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

Conversation

@cclauss

@cclauss cclauss commented Jan 31, 2019

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Collaborator

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

@cclauss

cclauss commented Feb 9, 2019

Copy link
Copy Markdown
Contributor Author

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

@scottleibrand

Copy link
Copy Markdown
Collaborator

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

@cclauss

cclauss commented Mar 10, 2019

Copy link
Copy Markdown
Contributor Author

@bewest

bewest commented Mar 11, 2019

Copy link
Copy Markdown
Member

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

cclauss commented Mar 11, 2019

Copy link
Copy Markdown
Contributor Author

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

@cclauss

cclauss commented Mar 12, 2019

Copy link
Copy Markdown
Contributor Author

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