-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
关于代码提交的一些建议。 #5
Comments
理解 |
正准备用nodejs 搞个 cms出来 管理移动端的数据 准备看看你的代码 有希望了 |
@mojun 项目已死。你可以搜索几个国外比较成型的框架学习。比如keystonejs |
一年前那个风雨交加的午后,小爝心血来潮…… |
keystonejs 太大 看不懂 我死得好惨啊 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
那天给许多人增加admin权限,可以直接pu上来。但是我不建议大家直接push。而是采取fork和pull request的方法来提交,我会及时查看提醒给大家merge。好处几点:
1,code review。
2,至少给你merge的人知道你pull上来的是什么功能,如果不清楚可以咨询我。防止破坏稳定版本。
3,每个fork都算是一个分支,可以互相不影响各自开发的功能。
随后我会开一个dev的分支作为开发分支,然后定期往master上合并。
The text was updated successfully, but these errors were encountered: