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

docs: remove line about Python 2.7 being unsupported #808

Merged
merged 2 commits into from
Nov 2, 2022

Conversation

rwhogg
Copy link
Contributor

@rwhogg rwhogg commented Oct 6, 2022

According to the README and setup.py, we do support Python 2.7

BEGIN_COMMIT_OVERRIDE
chore: remove line about Python 2.7 being unsupported
END_COMMIT_OVERRIDE

@rwhogg rwhogg requested review from a team and andrewsg as code owners October 6, 2022 19:13
@product-auto-label product-auto-label bot added the api: datastore Issues related to the googleapis/python-ndb API. label Oct 6, 2022
@rwhogg rwhogg force-pushed the remove-py2-unsupported-comment branch 5 times, most recently from 1cfc77a to 8b79bde Compare October 18, 2022 18:08
@leahecole leahecole added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 19, 2022
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 19, 2022
@meredithslota
Copy link

How does this change the code samples referenced above this line? https://github.com/googleapis/python-ndb/blob/main/docs/migrating.rst#ndbutils Are the utils fully functional in 2.7?

@rwhogg rwhogg force-pushed the remove-py2-unsupported-comment branch 2 times, most recently from 0649371 to 52ea0e0 Compare October 20, 2022 20:06
@rwhogg
Copy link
Contributor Author

rwhogg commented Oct 20, 2022

That code snippet will still work in both Py2 and Py3 (ignoring that it had a minor syntax error which I've now corrected).

Originally, @utils.positional raised NotImplementedError but that decorator was added back in in #203 .

I've adjusted the wording to reflect this.

rwhogg and others added 2 commits October 26, 2022 15:51
According to the README and setup.py, we do support Python 2.7.

Also clarify that the utils module and its positional decorator
do still exist and does work in both Py2 and Py3
@rwhogg rwhogg force-pushed the remove-py2-unsupported-comment branch from 21bc207 to ad3e9cc Compare October 26, 2022 19:51
@parthea parthea added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 2, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 2, 2022
@parthea parthea merged commit 06dea1b into googleapis:main Nov 2, 2022
@rwhogg rwhogg deleted the remove-py2-unsupported-comment branch November 2, 2022 14:50
@parthea parthea added the release-please:force-run To run release-please label Nov 16, 2022
@release-please release-please bot removed the release-please:force-run To run release-please label Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: datastore Issues related to the googleapis/python-ndb API.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants