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

Upgrade to POI 4.0.0 #80

Closed
wants to merge 5 commits into from
Closed

Upgrade to POI 4.0.0 #80

wants to merge 5 commits into from

Conversation

pjfanning
Copy link
Collaborator

I have forked the xlsx-streamer to support poi 4.0.0 but the APIs are basically the same.
The spoiwo library will probably also need to be upgraded when they publish a poi 4.0.0 compatible jar (they've already merged a change to support this).

@pjfanning pjfanning mentioned this pull request Sep 8, 2018
@nightscape
Copy link
Collaborator

Hi @pjfanning,

nice job, thanks for the contribution!
I would like to wait for your PR against xlsx-streamer to be merged, so that we don't switch dependencies and imports back and forth. Is that ok for you?

@pjfanning
Copy link
Collaborator Author

@nightscape the reason I published my own version of xlsx-streamer is because I suspect that the original project is not actively maintained.

@nightscape
Copy link
Collaborator

Rebased and merged in f775c72

@nightscape nightscape closed this Oct 11, 2018
@pjfanning pjfanning deleted the poi-4.0.0 branch October 11, 2018 15:21
@nightscape
Copy link
Collaborator

Released as 0.10.0. Will take a few hours to hit Maven Central though.

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