developit / developit/web-worker

Running through rollup for node causes issues with dynamic require

オープン
#6 コメント 11 件 リアクション 4 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
1.2k
フォーク
57
PR マージ指標
30日以内にマージされた PR はありません

説明

Thanks for a neat little library!

I was attempting at incorporating it into a library of mine which I bundle using rollup and I get an issue
````
Error: Dynamic requires are not currently supported by rollup-plugin-commonjs
````
which was related to [this line of your library](
https://github.com/developit/web-worker/blob/master/node.js#L190)

From what I can tell it looks like it's to do with requiring the URL of the worker?

Anyway just thought I'd flag it as other users of rollup may run into the same issue.

Cheers
Rowan

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。