browserify / browserify/static-module

Nested static module replacements don't work

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

説明

I think an example is the easiest way to explain this.

``` js
var staticModule = require('static-module');

var rs = require('stream').Readable();
rs.push("var foo = require('foo');\n");
rs.push("window.x = foo(1, foo(2, 3, 4), foo(5, 6, 7));\n");
rs.push(null);

var sm = staticModule({
foo: function (x, y, z) {
return 'bar(' + x + ', ' + y + ', ' + z + ')'
}
});

rs.pipe(sm).pipe(process.stdout);

// expected: bar(1, bar(2, 3, 4), bar(5, 6, 7));
// actual: bar(1, bar(2, 3, 4), bar(5, 6, 7))bar(2, 3, 4)bar(5, 6, 7);
```

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

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

issue の stream pipeline とネストした foo(...) の置換の例を再現に使用してください。期待される出力と実際の出力を比較し、各ネストした置換が一度だけ出力され、最後のセミコロンが保持されることを確認してください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, node.js
領域
tooling
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
35/100

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

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