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

Chap0x01 #1

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

Chap0x01 #1

wants to merge 4 commits into from

Conversation

cucsecmodan
Copy link
Collaborator

No description provided.

@yumlii33
Copy link

yumlii33 commented Sep 20, 2022 •
edited
Loading

Accepted

Pros

  • 符合推荐的目录和分支结构
  • 在PR标题中体现了作业序号
  • 见名知意的文件命名
  • 有基本的配图和高亮标记
  • 直接在配图中对高亮标记进行了细致的文字标注
  • 实验报告结构规范，内容完整
  • Markdown使用规范，代码块高亮
  • 参考文献/资料引用标准规范
  • 作业提交及时
  • 实验完成度较高，网络连通性测试基本完成 🎉
  • 绘制了本次实验中使用的网络拓扑图 👍
  • 对实验过程中遇到的问题和解决方法进行了记录 👍

Cons

  • ## 实验过程标题缺失
  • 内网靶机只需要添加一块intnet网卡。如果添加NAT网卡，靶机可以独立访问互联网，关闭网关主机后也可以访问互联网。
  • Line 56（举例）换行失败：Markdown可以通过行尾添加2个以上的空格或使用空行实现换行，详见：Markdown官网手册，建议学习一下~

Recommends

  • 文档中有一些多余的<br>可以不用添加~
  • Markdown规范的分点排版是- 作业要求，详见：Markdown官网手册，建议学习一下~
  • 可以善用分级标题、分级列表使得实验报告结构更加清晰、排版更加舒适~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
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
@cucsecmodan @yumlii33