LinusU / LinusU/node-jsonlines
Avoid use transform directly
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 25
- Forks
- 2
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
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 😄
Guide de contribution
Aucun guide de contribution indexé pour ce dépôt
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par localiser l’utilisation directe de Node's Transform API dans le module et examinez le manifeste du package pour connaître les conventions de dépendances. Remplacez cette utilisation par through2, puis vérifiez que l’analyse de JSON Lines fonctionne toujours et que le package est prêt pour la version v1.0.0 proposée.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- node.js
- Domaine
- backend
- Type d'issue
- Refactorisation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100