-
Notifications
You must be signed in to change notification settings - Fork 71
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
Added pre-commit hook and tests against python 3.5 and 3.6 #70
Added pre-commit hook and tests against python 3.5 and 3.6 #70
Conversation
docs/source/conf.py
Outdated
@@ -53,7 +54,7 @@ | |||
source_suffix = '.rst' | |||
|
|||
# The encoding of source files. | |||
#source_encoding = 'utf-8-sig' | |||
# source_encoding = 'utf-8-sig' |
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.
I'd recommend excluding docs/source/conf.py
, the formatting changes don't really make it more readable.
@@ -1,29 +1,28 @@ | |||
[tox] | |||
envlist = py27,py27-simplejson,py34 | |||
envlist = {py27,py35,py36}-{default,simplejson},pre-commit,cover,docs |
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.
Do we really need to test all of those Python versions with simplejson? Not knowing why this was introduced, I'd rather vote for dropping it, or are there known incompatibilities in recent versions?
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.
I'm defining all these environments too keep tox.ini "simple". On travis, we're running tests with simplejson only for python 2.7.
75f2deb
to
8f804db
Compare
8f804db
to
4390d8a
Compare
4390d8a
to
7f77afc
Compare
This CR introduces: