Skip to content

Update pako dependency to version ^2.0.2 for better tree shaking #24

@andreaslarssen

Description

@andreaslarssen

Describe your idea

From Hopding#689:

Angular11 generates warnings for pdf-lib because of the old module format of pako

Warning: C:\Users*****\node_modules\pdf-lib\es\core\PDFContext.js depends on 'pako'. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

Warning: C:\Users*****\node_modules@pdf-lib\upng\UPNG.js depends on 'pako'. CommonJS or AMD dependencies can cause optimization bailouts.
For more info see: https://angular.io/guide/build#configuring-commonjs-dependencies

Recently pako switched with version 2.0.0 to es6. Would it be possible to update the dependency?
Thank you!

How could this be implemented?

See above

What problem are you trying to solve?

See above

Why does this matter to you?

Optimization bailouts, and prevents us from upgrading to Angular 16 (?)

Would others find this helpful?

Yes

Are you interested in implementing your proposal?

No

Why are you submitting a proposal?

To keep the lib alive

Additional Notes

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    help wantedExtra attention is needed

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions