Skip to content
This repository has been archived by the owner on Sep 9, 2021. It is now read-only.

fix: make inline workers work from inside workers #307

Merged

Conversation

MatthewSteel
Copy link
Contributor

@MatthewSteel MatthewSteel commented Feb 10, 2021

This PR contains a:

  • bugfix (?)
  • new feature
  • code refactor
  • test update
  • typo fix
  • metadata update

Motivation / Use-Case

This change enables inline workers to be instantiated from worker scopes (where window isn't available.)

Breaking Changes

None.

Additional Info

fixes #306

@codecov
Copy link

codecov bot commented Feb 10, 2021

Codecov Report

Merging #307 (9cd816e) into master (edca167) will decrease coverage by 0.51%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #307      +/-   ##
==========================================
- Coverage   76.35%   75.83%   -0.52%     
==========================================
  Files           6        6              
  Lines         148      149       +1     
  Branches       51       52       +1     
==========================================
  Hits          113      113              
  Misses         30       30              
- Partials        5        6       +1     
Impacted Files Coverage Δ
src/runtime/inline.js 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update edca167...9cd816e. Read the comment docs.

Copy link
Member

@alexander-akait alexander-akait left a comment

Choose a reason for hiding this comment

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

Thanks

@alexander-akait alexander-akait merged commit 2abd129 into webpack-contrib:master Feb 11, 2021
TheLD6978 pushed a commit to TheLD6978/worker-loader that referenced this pull request Apr 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inline workers not working _in_ a worker
2 participants