1602 / 1602/compound

Consistent TypeOf

オープン 初心者向け
#627 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
1.6k
フォーク
179
PR マージ指標
30日以内にマージされた PR はありません

説明

The code has mixed style between `TypeOf something === function` and `function === TypeOf something`.

https://github.com/1602/compound/blob/master/lib/controller-extensions.js#L137
https://github.com/1602/compound/blob/master/lib/controller-extensions.js#L304
https://github.com/1602/compound/blob/master/lib/controller-extensions.js#L307
https://github.com/1602/compound/blob/master/lib/compound.js#L167
https://github.com/1602/compound/blob/master/lib/compound.js#L257
https://github.com/1602/compound/blob/master/lib/compound.js#L264

Most are `if (typeof VAR === 'function') {` style. So it wouldn't hurt to stick to it.

コントリビューションガイド

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

The issue points to specific lines in lib/controller-extensions.js and lib/compound.js. Open those files, find the typeof checks, and change any `function === typeof VAR` patterns to `typeof VAR === 'function'`. Run any existing tests to ensure the changes don't break functionality.

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, node.js
領域
backend, tooling
issue の種類
リファクタリング
難易度
1/5
見積もり時間
1時間未満
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
75/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。