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

Add Python 3.6 to CI #1821

Merged
merged 2 commits into from
Jan 24, 2017
Merged

Add Python 3.6 to CI #1821

merged 2 commits into from
Jan 24, 2017

Conversation

derekbekoe
Copy link
Member

Closes #1626

# PyLint does not yet support Python 3.6 https://github.com/PyCQA/pylint/issues/1241
if [ "$TRAVIS_PYTHON_VERSION" != "3.6" ]; then
check_style --ci;
fi
Copy link
Contributor

Choose a reason for hiding this comment

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

yeah, super annoy.

- Do not run pylint on 3.6 as pylint doesn’t support 3.6 yet
- Add Python 3.6 to package classifiers
@derekbekoe
Copy link
Member Author

@troydai can you take a look at the updated PR?

@troydai
Copy link
Contributor

troydai commented Jan 24, 2017

@derekbekoe On it.

@@ -46,6 +46,10 @@

# MOCK METHODS

# Workaround until https://github.com/kevin1024/vcrpy/issues/293 is fixed.
vcr_connection_request = vcr.stubs.VCRConnection.request
vcr.stubs.VCRConnection.request = lambda *args, **kwargs: vcr_connection_request(*args)
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm pretty sure flake8 will bad mouth this line of code. But we disable the style check here anyway.

@derekbekoe derekbekoe merged commit 8e1d2aa into Azure:master Jan 24, 2017
@derekbekoe derekbekoe deleted the python-3-6 branch January 24, 2017 03:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants