LinusU / LinusU/node-jsonlines
Avoid use transform directly
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 25
- Forks
- 2
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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
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 😄
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne damit, die direkte Verwendung von Node's Transform API im Modul zu lokalisieren und das Paketmanifest auf Abhängigkeitskonventionen zu prüfen. Ersetze diese Verwendung durch through2 und überprüfe anschließend, dass das Parsen von JSON Lines weiterhin funktioniert und das Paket für das vorgeschlagene v1.0.0-Release bereit ist.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- node.js
- Bereich
- backend
- Issue-Typ
- Refactoring
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100