Skip to content
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

merge Tommy APIJSON #33

Merged
merged 12 commits into from
Jun 17, 2018
Merged

merge Tommy APIJSON #33

merged 12 commits into from
Jun 17, 2018

Conversation

linlwqq
Copy link
Contributor

@linlwqq linlwqq commented Jun 14, 2018

No description provided.

@TommyLemon TommyLemon merged commit ed7b978 into Tencent:master Jun 17, 2018
@TommyLemon
Copy link
Collaborator

TommyLemon commented Jun 17, 2018

感谢贡献代码,已合并,现在你也是 APIJSON 项目的一个Contributer了
https://github.com/TommyLemon/APIJSON/graphs/contributors
e7fbe3a2-1db0-485b-b0ea-b3307ec54e44

但 NetsbdNews, NetsbdNewsCategory 只是你的业务代码,不具有通用性哦。
还有 .idea, target 等 编辑器或IDE 在打开工程后 [自动生成] 的关于配置或编译的缓存目录提交了没用的哦,每个人的都不一样,别人下载后再打开后会重新生成的。

@TommyLemon
Copy link
Collaborator

你fork出来APIJSON的分支最新版是 2017年11月18日 的,这大半年APIJSON有很多重要更新
https://github.com/TommyLemon/APIJSON/releases

建议发一个 你的仓库 linlwqq/APIJSON 从原仓库 TommyLemon/APIJSON 同步 的分支,这样就保持和原仓库同步了。

04cccddf-6788-4404-8810-41d7fe3af96e

看了下你的fork分支,我居然错误地把部分修改同步到你的仓库了,建议回滚下(如果需要以上那样同步,则先同步)。
https://github.com/linlwqq/APIJSON/commits/master

@TommyLemon
Copy link
Collaborator

TommyLemon commented Aug 31, 2018

@linlwqq 非常感谢,已把你加到APIJSON项目主页的 关于作者 一栏
https://github.com/TommyLemon/APIJSON

@TommyLemon
Copy link
Collaborator

@linlwqq 我打算把 APIJSON 放到自己任职公司的 GitHub 开源账号名下,吸引更多用户和贡献者,你觉得怎么样?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants