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

屎山爆破计划 #140

Open
2 tasks
Goodjooy opened this issue Jun 10, 2023 · 0 comments
Open
2 tasks

屎山爆破计划 #140

Goodjooy opened this issue Jun 10, 2023 · 0 comments

Comments

@Goodjooy
Copy link
Contributor

Goodjooy commented Jun 10, 2023

由于长时间维护,复杂的多个功能区合在一起。导致当前项目虽然已经具备了发布1.0的完整功能,但是代码本身质量依旧存疑。因此这里提出对于Rust端源码的大型重构。

首先,在开始重构前,希望能够提供全套的单元测试与集成测试,以确保在重构之后也能确认功能正确。

  • 占位符:添加 XXX 单元测试/集成测试,测试XXX 功能运作正常
  • 这里添加关于添加、修改、更新单元测试的 pull request

接下来,关于当前整个项目具有的缺陷,也许不同的人具有不同的看法,因此,可以在这里进行讨论。不管是新的issue 或者 在discussions 中发表

最后,开始重构吧。需要知道每一个pull request 对应的 issue

  • PR [ ]:完成Issue [ ] 内容要求的XXX 部分/全部
  • 这里列出所有PR 与 issue 关系,并添加引用
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

No branches or pull requests

1 participant