Skip to content
This repository was archived by the owner on May 4, 2020. It is now read-only.

Python 3 support#46

Merged
cjw296 merged 33 commits into
python-excel:masterfrom
takluyver:py3
Apr 13, 2015
Merged

Python 3 support#46
cjw296 merged 33 commits into
python-excel:masterfrom
takluyver:py3

Conversation

@takluyver
Copy link
Copy Markdown
Contributor

Following discussion on PR #32, this adds Python 3 support without any external dependencies. Python 2.6 and above and Python 3.3 and above should work; older versions will not.

The tests are passing on Python 2 and 3; I haven't tried the examples.

@takluyver takluyver mentioned this pull request Apr 26, 2014
@jreback
Copy link
Copy Markdown

jreback commented Apr 26, 2014

👍

@takluyver
Copy link
Copy Markdown
Contributor Author

@cjw296 : I just wanted to make sure this hasn't fallen off the radar. The tests appear to be passing.

@ghost
Copy link
Copy Markdown

ghost commented Jun 17, 2014

👍

Comment thread xlwt/antlr.py
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does losing the L have an impact on Py2?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

No, ints are automatically upgraded to longs if left-shifting takes them past maxint.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Okay.

@sontek
Copy link
Copy Markdown
Contributor

sontek commented Jan 20, 2015

To help with this I added a PR to this PR:

takluyver#1

That adds tox and pytest support and fixes bugs in a test that wasn't being ran by nose.

@sontek
Copy link
Copy Markdown
Contributor

sontek commented Jan 20, 2015

What is left to get this merged in? I'd love to see it :)

Add tox and pytest support to catch additional bugs
@sontek
Copy link
Copy Markdown
Contributor

sontek commented Jan 20, 2015

With takluyver#2 I've added the badges for pypi, coveralls, and travis. We should enable coveralls on this repo to get the badge working properly.

@sontek
Copy link
Copy Markdown
Contributor

sontek commented Jan 20, 2015

@cjw296 Beers on me at PyCon if you merge this :) 🍻 🍸

@collinanderson
Copy link
Copy Markdown

+1

@cjw296
Copy link
Copy Markdown
Member

cjw296 commented Jan 21, 2015

I'm coming to 3 days of sprints at PyCon with the express intent of merging this.
@sontek, hopefully you and @takluyver can be there to help?

@sontek
Copy link
Copy Markdown
Contributor

sontek commented Jan 21, 2015

@cjw296 we were hoping to get a release prior to pycon. We just ported all our stuff to py3 and want to have it fully in prod running stable releases before pycon.

Could we consider merging this to a develop/feature branch and release a prerelease version people can test out?

Are there specific things in this PR you feel aren't ready that we could help with?

@takluyver
Copy link
Copy Markdown
Contributor Author

I will be at Pycon, but I'd be happy to get this merged before then. I can think of more interesting things to be working on during the few days of sprints with Pythonistas from around the world.

@tjanez
Copy link
Copy Markdown

tjanez commented Mar 8, 2015

Hi!

Any progress on this?

@alexandre
Copy link
Copy Markdown

up

@collinanderson
Copy link
Copy Markdown

(btw if it helps, xlwt-future has py3 support, though beware it doesn't play will with newrelic)

cjw296 added a commit that referenced this pull request Apr 13, 2015
@cjw296 cjw296 merged commit 5e479b8 into python-excel:master Apr 13, 2015
@takluyver
Copy link
Copy Markdown
Contributor Author

Woo! 🎉 ✨ 🍰

@nicklasb
Copy link
Copy Markdown

I know what they fix, the blatant lack of back doors.
Woo!! . :-)

@cjw296
Copy link
Copy Markdown
Member

cjw296 commented Apr 13, 2015

@nicklasb - huh?

@nicklasb
Copy link
Copy Markdown

Just kidding.
Just wrt big commits that do more then they say and that I understand your apprehension. :-)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.