browserify / browserify/static-module

Nested static module replacements don't work

Abierto
#16 0 comentarios 1 reacción 0 asignados Ver en GitHub
bug
Lenguaje dominante
JavaScript
Estrellas
75
Forks
22
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

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);
```

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Usa como reproducción el ejemplo de reemplazo anidado de foo(...) y de la stream pipeline del issue. Compara la salida esperada con la real y, después, verifica que cada reemplazo anidado se emita una sola vez y que se conserve el punto y coma final.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
javascript, node.js
Área
tooling
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.