-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Feat/mfsu rebuild when patch #10065
base: master
Are you sure you want to change the base?
Feat/mfsu rebuild when patch #10065
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
2b4c8ca
to
0439e86
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## master #10065 +/- ##
=========================================
Coverage ? 30.36%
=========================================
Files ? 433
Lines ? 12431
Branches ? 2976
=========================================
Hits ? 3775
Misses ? 8089
Partials ? 567
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
首先非常感谢您一直认真的帮我找出代码规范的问题。看着每次 PR 都有那么多问题,找出来一定很辛苦,并且不是一个愉快的工作,我也觉得挺过意不去。但是目前自己认知水平达不到,在提交之前也看不出问题。 请问可以给一个您认可的编码规范的文档吗?虽然有些情形考虑不到,还是需要别人指点,但是我也希望提交的代码在代码风格上尽量别再有这么多的问题 =.= |
我是很久以前看过一些零散的文章,不过你可以去 juejin 搜一下 关于 code style ,我提的建议也是自己的看法,见仁见智,千人千面。 |
refactor: add second params
ca4d002
to
427de44
Compare
上次看你的提交,如果对 TypeScript 的类型不熟,可以用 这里还有一些题目可以看看:https://github.com/type-challenges/type-challenges |
support #10064