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

增加核心代码注释和文档 #165

Closed
hnwyllmm opened this issue May 8, 2023 · 1 comment
Closed

增加核心代码注释和文档 #165

hnwyllmm opened this issue May 8, 2023 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@hnwyllmm
Copy link
Collaborator

hnwyllmm commented May 8, 2023

Enhancement
增加核心模块代码注释和文档，可以帮助初学者更容易上手和理解
@hnwyllmm hnwyllmm added enhancement New feature or request good first issue Good for newcomers labels May 8, 2023
@hnwyllmm hnwyllmm mentioned this issue May 25, 2023
Merged
hnwyllmm added a commit that referenced this issue May 25, 2023
@hnwyllmm
Record manager doc (#185)
ac87eef 
### What problem were solved in this pull request?

Issue Number: ref #165 

Problem:
record_manager 没有比较详细的注释

### What is changed and how it works?
加一些注释

### Other information
@hnwyllmm hnwyllmm mentioned this issue Jun 14, 2023
Merged
hnwyllmm added a commit that referenced this issue Jun 14, 2023
@hnwyllmm
Refactor (#195)
93b79cc 
### What problem were solved in this pull request?

Issue Number: close #173 close #136 

ref #174 
ref #165 

Problem:
这个PR修改了多个问题，可以参考各个issue。包括observer编译两次、代码目录规划不合理、command executor未全部实现等

### What is changed and how it works?
做一些重构优化，包括CMakelist、一些代码目录、command executor
@hnwyllmm hnwyllmm mentioned this issue Jun 26, 2023
Merged
hnwyllmm added a commit that referenced this issue Jun 26, 2023
@hnwyllmm
增加一些文档 (#200)
f4e5213 
### What problem were solved in this pull request?

Issue Number: ref #165 
ref #174 

Problem:
一些文档需要优化
@hnwyllmm
Copy link
Collaborator Author

当前已经有了很多注释，后续的代码注释可以随相应的pull request提交

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@hnwyllmm