LinusU / LinusU/node-jsonlines
Avoid use transform directly
まだ誰も着手していません。
- 主要言語
- JavaScript
- スター
- 25
- フォーク
- 2
- PR マージ指標
- 30日以内にマージされた PR はありません
説明
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 😄
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
まず、モジュールでの Node's Transform API の直接使用箇所を特定し、依存関係の規約についてパッケージマニフェストを確認します。その使用を through2 に置き換えた後、JSON Lines のパースが引き続き機能することと、パッケージが提案された v1.0.0 リリースの準備ができていることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- node.js
- 領域
- backend
- issue の種類
- リファクタリング
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- おおむね明確
- 初心者へのやさしさ
- 35/100