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

feat(endo): Add zip #346

Merged
merged 1 commit into from
Jul 22, 2020
Merged

feat(endo): Add zip #346

merged 1 commit into from
Jul 22, 2020

Conversation

kriskowal
Copy link
Member

Our current archive format of choice is Zip. Our current implementation of choice is stuk/jszip. Endo's functions that interact with archives assume an Archive interface with read, write, data, and a from-data constructor. To decouple Endo's internals from the specifics of JSZip, this thin adapter presents the JSZip API as this Archive interface.

@kriskowal kriskowal mentioned this pull request Jun 19, 2020
36 tasks
@kriskowal kriskowal force-pushed the kris/endo-zip branch 2 times, most recently from 085d611 to ff0ff33 Compare June 19, 2020 19:34
@kriskowal kriskowal force-pushed the kris/endo-zip branch 4 times, most recently from f32e1ea to 24e8a1f Compare June 24, 2020 03:00
@kriskowal kriskowal force-pushed the kris/endo-zip branch 4 times, most recently from f325041 to a6078ed Compare June 30, 2020 19:18
@kriskowal kriskowal force-pushed the kris/endo-zip branch 2 times, most recently from 0926a2d to e87f75f Compare July 21, 2020 00:52
@kriskowal kriskowal force-pushed the kris/endo-zip branch 2 times, most recently from 7e6eb16 to 46a65c8 Compare July 21, 2020 20:26
Copy link
Contributor

@warner warner left a comment

Choose a reason for hiding this comment

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

looks good, although some day we might want a format that doesn't have to keep everything in RAM

Base automatically changed from kris/endo-import to master July 22, 2020 23:03
Our current archive format of choice is Zip.  Our current implementation of choice is stuk/jszip.  Endo's functions that interact with archives assume an Archive interface with read, write, data, and a from-data constructor.  To decouple Endo's internals from the specifics of JSZip, this thin adapter presents the JSZip API as this Archive interface.
@kriskowal
Copy link
Member Author

looks good, although some day we might want a format that doesn't have to keep everything in RAM

Indeed!

@kriskowal kriskowal merged commit 51c0867 into master Jul 22, 2020
@kriskowal kriskowal deleted the kris/endo-zip branch July 22, 2020 23:10
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.

2 participants