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

update #33

Closed
wants to merge 3 commits into from
Closed

update #33

wants to merge 3 commits into from

Conversation

AbstractDataType
Copy link
Contributor

@AbstractDataType AbstractDataType commented Jul 21, 2017

按照要求修改了一下。(不知道为什么core.php我没修改结果github也显示有修改...)

@oott123
Copy link
Owner

oott123 commented Jul 21, 2017

十分感谢你的 Pull Request。

你提到的没有改动却显示修改了的文件,是因为你将换行符(Line-ending)改为了 CRLF。可以参考教程来设置如何将 commit 的文件设为 LF ending。

这些修改我已经帮你清理了。其它的修改我也看过了,基本没有问题,谢谢你~

@oott123 oott123 closed this Jul 21, 2017
@oott123
Copy link
Owner

oott123 commented Jul 21, 2017

Merged at: a1948c4

@oott123 oott123 mentioned this pull request Jul 21, 2017
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.

None yet

2 participants