-
Notifications
You must be signed in to change notification settings - Fork 59
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
Refactor Plan #94
Comments
I'd like to have a try at refactoring package |
Thank you! I think you may refer to |
我其实不是很熟悉 DDD,目前大致重构了一部分(已经push到我自己的fork里了)。 |
首先,我认为DDD这个东西不必太过于在意它,现有的模块划分更多的只是为了避免循环调用,到底是不是教科书般的DDD或是DDD的最佳实践我觉得其实不是那么重要,能够比较舒服的让项目得到发展就可以了。 如果现在的工作告一段落,可以考虑先提一个PR,在PR中描述下做了什么。 |
了解了。已提 PR,有空的时候辛苦帮忙 Code Review 一下 |
There're some plan for refactor DouTok. If you have some advices, please comment here.
Also, welcome to select one or more entries to do something and that's DouTok's pleasure. If you want to make some contributions to DouTok, we suggest open another issue which used to tell everyone what you want to do and assign it to yourself. In the way, we can avoid different people doing the same job.
The following is something what we want to do. Absolutely, anything that does not appear below is acceptable.
Run & Build & Deploy
go run ./applications/xxx/
. We may use a better way to do these. See more on: usemake
to startup DouTok services #112CI/CD
go test
when open a PRCode Layout
Now the code of DouTok still not elegant enough. Code layout is a big problem. There many places in this repo is not OOP (although golang is not a 100% OOP language). We suggest that we can improve the code layout of this repo.
applications/commentDomain
is an example.Some package may need refactor:
applications/feed
applications/message
applications/messageDomain
applications/publish
applications/relationDomain
@xwxbUnit Test
Unfortunately, DouTok is poor on unit test. So any contributions for any package are welcome.
The text was updated successfully, but these errors were encountered: