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

chap0x04实验报告提交 #4

Open
wants to merge 1 commit into
base: chap0x04
Choose a base branch
from
Open

Conversation

ZXMMD
Copy link
Collaborator

@ZXMMD ZXMMD commented May 8, 2020

chap0x04实验报告提交

@TheMasterOfMagic
Copy link

pr里只有一个.travis.yml文件?🤔

@ZXMMD
Copy link
Collaborator Author

ZXMMD commented May 11, 2020

助教师哥您好,我是按老师说的,fork了自己的作业仓库,在自己的个人仓库里完成travis操作后,从自己的个人仓库发起了PR。我把实验报告和代码及图片都上传到了作业仓库,然后才fork,在个人仓库完成travis操作。我的实验报告和代码都在作业仓库里。我应该是理解错了老师的意思,应该先fork,然后在个人仓库里完成所有操作。请问我该如何补救呢?(我还在语雀里问老师用不用从自己的作业仓库发起PR,老师说不用。我以为您能看到我的实验报告呢)

@c4pr1c3
Copy link

c4pr1c3 commented May 11, 2020

助教师哥您好,我是按老师说的,fork了自己的作业仓库,在自己的个人仓库里完成travis操作后,从自己的个人仓库发起了PR。我把实验报告和代码及图片都上传到了作业仓库,然后才fork,在个人仓库完成travis操作。我的实验报告和代码都在作业仓库里。我应该是理解错了老师的意思,应该先fork,然后在个人仓库里完成所有操作。请问我该如何补救呢?(我还在语雀里问老师用不用从自己的作业仓库发起PR,老师说不用。我以为您能看到我的实验报告呢)

你确实理解错了我的期望作业提交模型和步骤。我期望的是:

  1. 不在作业仓库里新建 chap0x04 分支;
  2. 直接通过个人账号 fork 自己的作业仓库;
  3. 在自己的个人仓库下新建 chap0x04 分支,本次作业直接提交到自己个人仓库的 chap0x04 分支和独立子目录;
  4. 作业完成后,在作业仓库发起 PR:请求从自己个人仓库的 chap0x04 分支合并到作业仓库的 chap0x04 或 master 分支都可以,但千万不要执行 Merge 操作。

经过以上操作后,我们在批改作业的时候,还是去你的作业仓库 PR 里看 Files Changed ,然后去你个人仓库的对应作业分支找到 travis build 链接查看「在线执行结果」。

@c4pr1c3
Copy link

c4pr1c3 commented May 11, 2020

助教师哥您好,我是按老师说的,fork了自己的作业仓库,在自己的个人仓库里完成travis操作后,从自己的个人仓库发起了PR。我把实验报告和代码及图片都上传到了作业仓库,然后才fork,在个人仓库完成travis操作。我的实验报告和代码都在作业仓库里。我应该是理解错了老师的意思,应该先fork,然后在个人仓库里完成所有操作。请问我该如何补救呢?(我还在语雀里问老师用不用从自己的作业仓库发起PR,老师说不用。我以为您能看到我的实验报告呢)

不用补救了, @TheMasterOfMagic 辛苦一下,直接一路链接点过去看 构建结果 ,第4章作业直接看他个人仓库的对应分支就好了。

@TheMasterOfMagic
Copy link

👌

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

  • 没什么大问题

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.

3 participants