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

Replaced vendored openpyxl by a dependency #217

Closed
wants to merge 3 commits into from

Conversation

claudep
Copy link
Contributor

@claudep claudep commented Feb 8, 2016

Thanks Tommy Anthony for the initial patch.

@claudep
Copy link
Contributor Author

claudep commented Feb 8, 2016

WIP: update method for syncing tablib.core.__version__ and setup.py's setup version param.

Importing tablib in setup.py to read the version number is problematic when
dependencies are needed during tablib import.
Recent dependencies are dropping Python 3.2 too.
Thanks Tommy Anthony for the initial patch.
@claudep
Copy link
Contributor Author

claudep commented Feb 10, 2016

Seems this is ready for review.

@kennethreitz
Copy link
Contributor

@claudep can you open this again against the master branch? sorry, doing some repo cleanup.

@claudep
Copy link
Contributor Author

claudep commented Feb 16, 2016

Sure, but no need to close the pull request and create another one, I can rebase my patch and force-push it. Could you reopen?

@kennethreitz
Copy link
Contributor

I deleted the develop branch, which this PR was made against. GitHub automatically closes the PR in that case, apparently. Sorry for the inconvenience, I didn't know :)

@claudep
Copy link
Contributor Author

claudep commented Feb 16, 2016

Yes, there was something strange with that develop branch. I opened #221, hopefully against the right branch!

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.

None yet

2 participants