-
-
Notifications
You must be signed in to change notification settings - Fork 380
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
Travis CI: Add Python 3.7 to the testing #215
Conversation
Python 3.7 on Travis CI added in alignment with travis-ci/travis-ci#9069
Thanks for PR @cclauss, several notes
|
.travis.yml
Outdated
script: | ||
- if [[ ${TRAVIS_SECURE_ENV_VARS} = false ]]; then | ||
echo "DISABLE INTEGRATION TESTING FOR S3"; | ||
unset SO_DISABLE_MOCKS; | ||
unset SO_S3_URL; | ||
unset SO_S3_RESULT_URL; | ||
fi | ||
- if [[ $TRAVIS_PYTHON_VERSION = '*.7' ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@cclauss no need to add flake here, this will be added with other changes & code cleanup later
Boto's own tests seem to pass on Python 3.7 boto/boto#3823 The env vars changes did not fix things... Removing the flake8 check. |
@cclauss anyway, need to fix this issue with boto first before the merge |
@cclauss any ideas how to fix this? |
Already added by #240, closing |
Python 3.7 on Travis CI added in alignment with travis-ci/travis-ci#9069