Skip to content

Updating Code to Use Classes #163

@MendyBerger

Description

@MendyBerger

Some of the yauzl objects still use the legacy util.inherits to extend classes. Would you be open to merging a PR that changes those to es6 classes with extend?

Why do I need this?

I'm trying to use yauzl in a non-node environment, and use unenv to polyfill the node APIs. unenv uses es6 classes in their polyfill, and when yauzl tries extending these classes with util.inherits, it fails since es6 classes can't be extended this way.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions