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

chapter0x01报告提交 #1

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

chapter0x01报告提交 #1

wants to merge 2 commits into from

Conversation

oldmovie
Copy link
Collaborator

chapter0x01报告提交

@yumlii33
Copy link

Accepted

Pros

  • 符合推荐的目录和分支结构
  • 在PR标题中体现了作业序号
  • 见名知意的文件命名
  • 有基本的配图和高亮标记
  • 实验报告结构规范
  • Markdown使用规范
  • 参考文献/资料引用标准规范
  • 作业提交及时
  • 实验完成度较高,网络连通性测试基本完成 🎉

Cons

  • 部分图片渲染失败,例如target_accesses the_attacker.png,建议提交前自查!
  • 代码段没有高亮
  • 记得按时提交作业

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.

2 participants