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 #10

Merged
merged 7 commits into from
Dec 1, 2017
Merged

Add python 3.6 #10

merged 7 commits into from
Dec 1, 2017

Conversation

gdalmau
Copy link
Contributor

@gdalmau gdalmau commented May 25, 2017

Closes #9

This PR is to make the library compatible with python 3.6

@gdalmau gdalmau modified the milestone: v0.0.3 May 25, 2017
@@ -15,7 +15,7 @@ def run(self):
"""Comença la tasca de neteja."""
_clean.run(self)
if os.path.exists(self.build_base):
print "Cleaning %s dir" % self.build_base
print("Cleaning {} dir".format(self.build_base))
Copy link
Member

Choose a reason for hiding this comment

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

Això no et funcionarà amb python2.7 si no fas un import que sigui:

from __future__ import print_statement

@coveralls
Copy link

coveralls commented May 25, 2017

Coverage Status

Changes Unknown when pulling b92e55c on test_python_3.6 into ** on master**.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.299% when pulling 3fbce51 on test_python_3.6 into 1aaf55c on master.

@gdalmau gdalmau requested a review from ecarreras May 25, 2017 14:56
@ecarreras
Copy link
Member

For now I will change that tests on py3 not are mandatory adding this to the .travis.yml file:

matrix:
  fast_finish: true
  allow_failures:
    - python: "3.6"

@coveralls
Copy link

Coverage Status

Coverage remained the same at 81.299% when pulling 893294e on test_python_3.6 into 1aaf55c on master.

@gdalmau
Copy link
Contributor Author

gdalmau commented May 31, 2017

@ecarreras I wouldn't close this PR because the purpose of it is to make the library working with python3.6.

@gdalmau gdalmau removed this from the v0.0.3 milestone Aug 10, 2017
@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.175% when pulling 094d152 on test_python_3.6 into fd44039 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.14% when pulling a5c8cb7 on test_python_3.6 into 49df158 on master.

@ecarreras ecarreras merged commit d456bef into master Dec 1, 2017
@ecarreras ecarreras deleted the test_python_3.6 branch December 1, 2017 10:14
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.

None yet

4 participants