Skip to content

Conversation

@vcfriend
Copy link

No description provided.

@vcfriend
Copy link
Author

添加注释

@wukan1986
Copy link
Contributor

我看了一下,发现你的代码存在以下问题:
1.提交了很多与代码无关的文件
2.与原有代码整文件整文件的有区别,但实际上代码内容并没有修改。
3.有些代码本人已经改动了很多,但你还是基于老版本改的

另外pull request时最好一个任务一个任务的pull request,其实有些代码想通过,但因为夹杂着很多不相关的内容,导致全都无法通过。

@vcfriend
Copy link
Author

vcfriend commented Jul 4, 2014

谢谢伍侃大哥的指教,我刚接触GitHub,用的还不太熟练。您的支持是我前进最大的动力。
Date: Tue, 17 Jun 2014 18:03:58 -0700
From: notifications@github.com
To: OpenQuant@noreply.github.com
CC: vcfriend@live.cn
Subject: Re: [OpenQuant] 添加注释。 (#3)

我看了一下,发现你的代码存在以下问题:

1.提交了很多与代码无关的文件

2.与原有代码整文件整文件的有区别,但实际上代码内容并没有修改。

3.有些代码本人已经改动了很多,但你还是基于老版本改的

另外pull request时最好一个任务一个任务的pull request,其实有些代码想通过,但因为夹杂着很多不相关的内容,导致全都无法通过。


Reply to this email directly or view it on 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