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

Project4 #4

Open
wants to merge 2 commits into
base: Project4
Choose a base branch
from
Open

Project4 #4

wants to merge 2 commits into from

Conversation

MN962464
Copy link
Collaborator

@MN962464 MN962464 commented May 9, 2020

第四章作业提交

Copy link

@TheMasterOfMagic TheMasterOfMagic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

Basis

  • 实验完成度:较高
  • markdown使用习惯:较好
  • git使用规范:较好
  • 报告规范度:一般
  • 提交时间:略晚

Recommends

  • 你的pr里也没有实验报告呀?可以看看老师在这份作业下的评论,你估计也是理解错老师意思了
  • 虽然我能看见你的实验数据文件,不过该在实验报告里给的链接还是该给的哈。

@MN962464
Copy link
Collaborator Author

不好意思助教师哥,因为我是现在作业仓库commit了实验报告等数据,又fork了个人仓库调试Travis文件,所以pr时不能pr实验报告等文件,但在我的个人仓库里都有,可能要麻烦师哥再看一下,或者还需要我把作业仓库里的删掉再pr一次吗?

@TheMasterOfMagic
Copy link

不好意思助教师哥,因为我是现在作业仓库commit了实验报告等数据,又fork了个人仓库调试Travis文件,所以pr时不能pr实验报告等文件,但在我的个人仓库里都有,可能要麻烦师哥再看一下,或者还需要我把作业仓库里的删掉再pr一次吗?

不用啦,我看到了的,就是提醒你一下。

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