-
Notifications
You must be signed in to change notification settings - Fork 350
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
task3: JeremyZ #919
task3: JeremyZ #919
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
task3 的 PR,请不要对其他 task 的文件进行修改,谢谢
这边没有修改其他task的文件,因为之前修改了文件夹名称,为了防止冲突,我rebase了task1的分支,导致看到的commit是这样的,task1修改可以参照下面: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
“为了防止冲突,我rebase了task1的分支”
说明你还是没有明白如何正确提交,请去 B 站多看几遍 欧雷 关于提交 PR 的视频,谢谢
我为了task1和task3并行,所以在task1修改文件夹名称后,task3 rebase了task1的分支,这样子我task3的修改也是在新文件夹名上修改的,merge之后才不会导致文件夹名称的冲突,如果真的需要,我就直接从task1新建分支我重新开一个merge吧 |
没新开PR的原因是因为我怕PR开启时间不对影响最后的任务奖励判断 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
提交 PR 的时间,不会影响(只要在期限之内提交,都一样);能否正确的提交 PR,才是真正会影响的。
新开了PR:#1078 |
Task3
由于Task1 PR修改了文件夹名,因此需要先对task1 PR进行合并!!!
Task1: #517