Skip to content
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

Update documentation for Python 3 #435

Merged
merged 2 commits into from
Dec 30, 2021
Merged

Conversation

gschizas
Copy link
Contributor

Given that Python 2 is out of support, the documenation examples should probably use Python 3 syntax (i.e. print as a function, input instead of raw_input) instead of Python 2 syntax.

Given that Python 2 is out of support, the documenation examples should probably use Python 3 syntax (i.e. print as a function, input instead of raw_input) instead of Python 2 syntax.
@coveralls
Copy link

coveralls commented Jan 16, 2021

Coverage Status

Coverage increased (+0.08%) to 90.239% when pulling e58c004 on gschizas:patch-1 into 46f886c on requests:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 90.164% when pulling 5fcebf0 on gschizas:patch-1 into 46f886c on requests:master.

Copy link
Contributor

@JonathanHuot JonathanHuot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM Thanks for the update

@JonathanHuot JonathanHuot added this to the 1.3.1 milestone Nov 2, 2021
Copy link
Contributor

@jtroussard jtroussard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm will this trigger ci/cd - dont think so just trying

@jtroussard
Copy link
Contributor

jtroussard commented Dec 30, 2021

Made a tiny whitespace change to trigger the new coveralls/github actions pipeline.

@jtroussard jtroussard merged commit ee20855 into requests:master Dec 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants