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.

Sign up for GitHub

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

Jump to bottom

Update 无人值守安装镜像实验报告.md #1

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update 无人值守安装镜像实验报告.md #1

wants to merge 3 commits into from

Conversation

Stephaniesuu
Copy link
Collaborator

更改分支

@huangyifei226 huangyifei226 self-requested a review March 21, 2021 14:09
huangyifei226
huangyifei226 approved these changes Mar 21, 2021
View reviewed changes
Copy link

@huangyifei226 huangyifei226 left a comment

Choose a reason for hiding this comment

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

Accepted

Pros

  • 实验报告内容较为完整、叙事逻辑清晰；
  • 懂得在作业中适时「植入自己的身份标识」来证明自己作业的原创性；
  • 见名知意的图片命名；
  • 在实验报告末总结了遇到的问题与解决方案；
  • 知道利用工具而不是肉眼方式进行文本比对👍

Recommends

  • 以后作业的完成时间尽可能在一周之内（下一周上课之前）完成并提交 PR；
  • 文本比对工具，如果是 CLI 方式推荐 vimdiff 。如果是 GUI 方式推荐 meld；
  • 实验报告有几处换行失败了；
  • 实验报告的截图中的关键信息，比如输入的命令，有的地方用红方框框出来了，看起来就很清晰，建议其他的截图里面也框出来哦；

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@huangyifei226 huangyifei226 huangyifei226 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Stephaniesuu @huangyifei226