you-dont-need/You-Dont-Need-Lodash-Underscore

Add more suggestions for _.first

Open

#312 opened on 2021年5月2日

GitHub で見る
 (1 comment) (1 reaction) (0 assignees)JavaScript (811 forks)batch import
enhancementhelp wanted

Repository metrics

Stars
 (18,379 stars)
PR merge metrics
 (30d に merged PR はありません)

説明

Currently, the suggestion we get for _.first might not be relevant in all cases. Array.prototype.slice() returns an array. But _.first from lodash returns a non-array value. So this line should be something like this I guess:

"alternative": "Array.prototype.slice() or arr[0]"

Also, I think we should add more aliases here. For example _.head should be prevented when _.first is prevented, because they're basically the same.

コントリビューターガイド

調査方針
ファイル lib/rules/rules.json の 51 行目を特定します。 .first の alternative フィールドを修正し、Array.prototype.slice() に加えて 'arr[0]' を含めます。また、 .head などのエイリアスを追加して、 .first が防止される際にそれらも防止されるようにします。ルールファイル内でのエイリアス追加の既存パターンを確認してください。
技術スタック
javascript
領域
documentation
Issue 種別
機能
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1時間未満
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
古い
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
basic JavaScriptJSON formatfamiliarity with lodash
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
70