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

Python 3.7 support #1733

Merged
merged 3 commits into from
Sep 22, 2018
Merged

Python 3.7 support #1733

merged 3 commits into from
Sep 22, 2018

Conversation

yan12125
Copy link
Contributor

Fixes #1706

.travis.yml is so complicated as Python 3.7 is not fully supported on Travis CI. See travis-ci/travis-ci#9815

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 92.307% when pulling fc3292d09d8ed2db0db773c766241b8c27769487 on yan12125:py37 into a1d095c on spulec:master.

@coveralls
Copy link

coveralls commented Jul 24, 2018

Coverage Status

Coverage remained the same at 92.413% when pulling 5224858 on yan12125:py37 into 9c6e7f2 on spulec:master.

@yan12125
Copy link
Contributor Author

All tests finally pass. Networking on Travis CI VMs is not so stable...

.travis.yml Outdated
@@ -8,6 +8,16 @@ python:
env:
- TEST_SERVER_MODE=false
- TEST_SERVER_MODE=true
matrix:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you not add 3.7 into the python: list above?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It doesn't work - https://travis-ci.org/spulec/moto/builds/407918370.

Maybe I should add a comment to remind future visitors about the relevant Travis CI issue?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Wow that's odd. Yeah add a comment, and if you dont mind, poke travis to see if they're aware of that as I'd think that should "just work"

Copy link
Contributor Author

@yan12125 yan12125 Jul 26, 2018

Choose a reason for hiding this comment

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

Updated!

Yes they are aware - the "sudo + xenial" solution comes from a Travis staff - travis-ci/travis-ci#9815 (comment). Sigh.

@terricain
Copy link
Collaborator

other than that comment it looks ok

@terricain
Copy link
Collaborator

@JackDanger you dont mind testing on 3.7 right?

@yan12125 yan12125 closed this Jul 26, 2018
@yan12125 yan12125 reopened this Jul 26, 2018
@yan12125 yan12125 closed this Jul 27, 2018
@yan12125 yan12125 reopened this Jul 27, 2018
@lalithakolla
Copy link

is there any reason this is not merged yet ?? it would be nice to use python3.7

@yan12125
Copy link
Contributor Author

yan12125 commented Aug 2, 2018

@lalithakolla: I believe @JackDanger's approval is required.

@terricain
Copy link
Collaborator

No so much required more a second opinion would be nice, worse case in a few days I'll merge.

@yan12125
Copy link
Contributor Author

yan12125 commented Aug 6, 2018

worse case in a few days I'll merge.

Seems @JackDanger is too busy to check this PR. How about just going ahead?

@yan12125 yan12125 mentioned this pull request Aug 6, 2018
@yan12125
Copy link
Contributor Author

Any news? I rebased to the latest master and it still looks good.

@spulec spulec merged commit 74b7634 into getmoto:master Sep 22, 2018
@spulec
Copy link
Collaborator

spulec commented Sep 22, 2018

Looks great; thanks!

@yan12125 yan12125 deleted the py37 branch September 23, 2018 02:45
@yan12125
Copy link
Contributor Author

Thanks! But seems the fix is not included in version 1.3.6?

And here's something strange for version 1.3.6 - the latest commit on https://github.com/spulec/moto/commits/master is 57f9691, but the 1.3.6 tag is on 4227cfd

@jonafato
Copy link
Contributor

Echoing @yan12125's comment here. Any chance of a stable release with 3.7 support soon?

(Thanks for building this!)

jonafato added a commit to jonafato/moto that referenced this pull request Dec 29, 2018
As of getmoto#1733, Python 3.7 is supported, so reflect that in the Trove
classifiers.

As of 2017-09-29, Python 3.3 is end-of-life and no longer receives
updates of any kind (including security fixes), so remove it from the
list of supported versions.
major0 added a commit to major0/salt that referenced this pull request Oct 8, 2020
A large number of the boto unit tests (roughy 1/3rd) were disabled in
the past as a result of getmoto/moto#1706 which
appears to have been fixed with getmoto/moto#1733

This commit enables the previously disbaled unit tests.
dwoz pushed a commit to saltstack/salt that referenced this pull request Oct 8, 2020
A large number of the boto unit tests (roughy 1/3rd) were disabled in
the past as a result of getmoto/moto#1706 which
appears to have been fixed with getmoto/moto#1733

This commit enables the previously disbaled unit tests.
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.

6 participants