Skip to content

Conversation

@CooooolFrog
Copy link
Contributor

@CooooolFrog CooooolFrog commented Jul 2, 2025

Background

Provide an overall design document of dragonfly to help new developers quickly understand the system's modules and design, and reduce the cost of understanding the code.

@CooooolFrog CooooolFrog requested a review from a team as a code owner July 2, 2025 10:38
mingcheng
mingcheng previously approved these changes Jul 7, 2025
Copy link
Member

@mingcheng mingcheng left a comment

Choose a reason for hiding this comment

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

LGTM

@mingcheng
Copy link
Member

Hi, You should resolve the build error: Docs markdown link couldn't be resolved

@CooooolFrog CooooolFrog force-pushed the add_system_design_docs branch 2 times, most recently from 010caa3 to bb6d170 Compare July 10, 2025 01:47
@CooooolFrog
Copy link
Contributor Author

Hi, You should resolve the build error: Docs markdown link couldn't be resolved

Thanks, there was a problem with the format of this link, I have fixed it now.

@CooooolFrog CooooolFrog force-pushed the add_system_design_docs branch 4 times, most recently from 4fa0d94 to cb829f7 Compare July 10, 2025 03:41
@CooooolFrog CooooolFrog changed the title [WIP] docs: add system design documentation docs: add system design documentation Jul 18, 2025
@CooooolFrog CooooolFrog requested a review from mingcheng July 21, 2025 03:06
@CooooolFrog CooooolFrog force-pushed the add_system_design_docs branch from cb829f7 to a5d9976 Compare August 1, 2025 02:30
@CooooolFrog CooooolFrog force-pushed the add_system_design_docs branch from a5d9976 to 30da046 Compare August 1, 2025 02:34
@CooooolFrog CooooolFrog requested a review from gaius-qi August 1, 2025 02:36
Copy link
Member

@gaius-qi gaius-qi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@LunaWhispers LunaWhispers left a comment

Choose a reason for hiding this comment

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

lgtm

@LunaWhispers LunaWhispers merged commit e97f37d into dragonflyoss:main Aug 26, 2025
5 checks passed
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.

4 participants