max-mapper / max-mapper/extract-zip
Pls add support for electron original-fs, as ADM-Zip does
Open
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 398
- Forks
- 144
- PR merge metrics
- No merged PRs in 30d
Description
https://www.npmjs.com/package/adm-zip
adm-zip add support to distingish electron environment , which has its own patch on fs.
Please refer:
https://github.com/electron/electron/issues/2035
https://github.com/electron/electron/issues/2198
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reading the ADM-Zip package linked in the issue and the referenced Electron issues about original-fs. Then inspect extract-zip's filesystem entry points and existing tests. Done means extraction works correctly in Electron's patched filesystem environment, with coverage for that behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- electron, javascript
- Domain
- desktop-dev, tooling
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100