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

Support for gzip/bz2 remote sources #463

Merged
merged 7 commits into from Jan 4, 2019

Conversation

@H-Max
Copy link
Contributor

commented Dec 28, 2018

Added support for opening remote gzip/bz2 files directly.

Support for URLSource or GZipSouce/BZ2Source was fine, but PETL could not open a remote gzip file directly (which is quite handy).

H-Max added 5 commits Feb 2, 2016
Merge pull request #1 from alimanfoo/master
Upgrade 1by1 repository to 1.1.0
Merge pull request #3 from petl-developers/master
Merge PETL master into 1by1 fork
@coveralls

This comment has been minimized.

Copy link

commented Dec 28, 2018

Coverage Status

Coverage decreased (-0.06%) to 91.171% when pulling 9bae9e6 on Marketing1by1:master into 2cb5dc5 on petl-developers:master.

@alimanfoo alimanfoo added this to the v1.3 milestone Jan 4, 2019

@alimanfoo
Copy link
Collaborator

left a comment

Many thanks @H-Max, this seems like a pragmatic solution, happy to go with it.

alimanfoo added 2 commits Jan 4, 2019

@alimanfoo alimanfoo merged commit 3d4c5e8 into petl-developers:master Jan 4, 2019

0 of 2 checks passed

continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
@H-Max

This comment has been minimized.

Copy link
Contributor Author

commented Jan 6, 2019

Glad to help !

Also, there is a type in the release notes. I think we should read:

Added support for reading from remote....

And can you mention my personnal username "H-Max" rather than my company ?

@alimanfoo

This comment has been minimized.

Copy link
Collaborator

commented Jan 6, 2019

Sorry for the mistakes in the release notes, should be fixed now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.