apache / apache/cordova-node-xcode

Cleanup

オープン
#78 コメント 2 件 リアクション 0 件 担当者 0 名 GitHub で見る
enhancement help wanted
主要言語
JavaScript
スター
217
フォーク
102
平均マージ
2日 3時間
マージ済み PR(30日)
6

説明

From review of PR #56 (watch2 support), I have a feeling that we should be able to clean up both the source code and all of the tests. I am raising this for future consideration and discussion.

Starting to add a list:

- [ ] use const & let instead of var, especially in new code (noticed after merging PR #56)
- [ ] look for ways to make the code easier to read & understand, especially the code added in PRs #56 & #76

/cc @l3ender

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

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

調査の方向性

まず PR #56 と PR #76 を確認し、次にそれらの変更の影響を受けるソースコードとテストを調査します。大まかなクリーンアップの依頼を、可読性と var-to-const/let に関する具体的なタスクに落とし込み、各タスクで対象のコードとテストを特定し、完了時に期待される状態を定義します。

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

評価

技術スタック
javascript, node.js
領域
tooling
issue の種類
リファクタリング
難易度
5/5
見積もり時間
1週間以上
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
20/100

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

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