Skip to content

Conversation

@wangxumin
Copy link

No description provided.

@tingxins
Copy link
Owner

tingxins commented Dec 3, 2017

Review 了下代码,功能基本没什么问题,下一个迭代版本可合并,但还有几处需修改一下哈:

  • 此 PR 修改了工程相关配置
  • Commit message 格式没对齐(可参考历史提交记录)
  • 不同功能建议分多次 PR,包括 Demo 的修改也是。

感谢支持。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants