apache / apache/cordova-node-xcode

Cleanup

未关闭
#78 2 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
enhancement help wanted
主要语言
JavaScript
星标
217
派生
102
平均合并
2 天 3 小时
30 天内合并 PR
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
预计耗时
一周以上
活跃度
停滞
描述清晰度
需要澄清
新手友好度
20/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。