Skip to content

Comments

Re-add note about Python 3 support#151

Merged
thunterdb merged 1 commit intographframes:masterfrom
nchammas:patch-1
Jan 4, 2017
Merged

Re-add note about Python 3 support#151
thunterdb merged 1 commit intographframes:masterfrom
nchammas:patch-1

Conversation

@nchammas
Copy link
Contributor

I talked @mengxr out of adding this in the first time around here in this PR, but it looks like GrapheFrames works fine with Python 3, and I've been using it as such for many weeks now.

@codecov-io
Copy link

Current coverage is 87.18% (diff: 100%)

Merging #151 into master will not change coverage

@@             master       #151   diff @@
==========================================
  Files            18         18          
  Lines           765        765          
  Methods         687        687          
  Messages          0          0          
  Branches         68         68          
==========================================
  Hits            667        667          
  Misses           98         98          
  Partials          0          0          

Powered by Codecov. Last update 775367b...88e4564

@mengxr
Copy link
Contributor

mengxr commented Dec 21, 2016

@nchammas I think we cannot promise python 3 support without testing it in Travis CI. Do you have time to update the Travis file?

@nchammas
Copy link
Contributor Author

Good point. Will do.

@nchammas
Copy link
Contributor Author

@mengxr - It looks like tests are already being run against Python 3.

@thunterdb thunterdb merged commit be6f86b into graphframes:master Jan 4, 2017
@nchammas nchammas deleted the patch-1 branch January 4, 2017 03:28
jkbradley pushed a commit that referenced this pull request Jan 17, 2017
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.

4 participants