Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Make worker-loader optional as peerDependencies. Close #13825. #13833

Merged

Conversation

tamuratak
Copy link
Contributor

Copy link
Collaborator

@Snuffleupagus Snuffleupagus left a comment

Choose a reason for hiding this comment

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

Please fix the lint failures, see https://github.com/mozilla/pdf.js/pull/13833/files or https://github.com/mozilla/pdf.js/pull/13833/checks?check_run_id=3211936793; you could (most likely) even do that automatically by running gulp lint --fix locally.

@tamuratak tamuratak force-pushed the add_peer_dep_meta_for_worker_loader branch from 25e101b to 9eca5e5 Compare August 1, 2021 08:45
Copy link
Collaborator

@Snuffleupagus Snuffleupagus 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, thank you!

@Snuffleupagus Snuffleupagus merged commit 4483cba into mozilla:master Aug 1, 2021
@tamuratak tamuratak deleted the add_peer_dep_meta_for_worker_loader branch November 13, 2023 03:44
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should use peerDependenciesMeta for worker-loader
2 participants