LinusU / LinusU/node-jsonlines
Avoid use transform directly
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 25
- Forks
- 2
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Descripción
Hey,
I'm seeing you are using `Transform` directly from Node API.
Event this is currently working, could be don't work if the next Node version change the node version.
Then your module will be broken.
A better approach is use Transform as dependency using [through2](https://www.npmjs.com/package/through2)
In fact, it was created exactly for this reason, I recommend you read this author blog post:
https://r.va.gg/2014/06/why-i-dont-use-nodes-core-stream-module.html
PS: Could be a good opportunity to use it and ship v1.0.0 😄
Guía de contribución
No hay ninguna guía de contribución indexada para este repositorio
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza por localizar el uso directo de Node's Transform API en el módulo y revisar el manifiesto del paquete para conocer las convenciones de dependencias. Sustituye ese uso por through2 y verifica después que el análisis de JSON Lines siga funcionando y que el paquete esté listo para la versión v1.0.0 propuesta.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- node.js
- Área
- backend
- Tipo de issue
- Refactorización
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100