-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Feature] 对 PR 代码评审中存在问题的一些总结和改进建议,供参考 #9136
Comments
很好的建议,RTT是逐步发展起来的,希望可以建立好的管理制度。也希望loop更多人一起来把关,例如分级、分子系统的maintainer制度 |
我觉得 问题5 中的 maintainer 制度是否可以优先建立起来,只有找对合适的,负责任的人,并逐级推广之,其他问题都能慢慢解决。 |
good idea,但现实情况现在开源社区贡献的人,能力参差不齐,有能力把关且愿意参与进来的更少,需要跟多无私奉献的。 |
是的,国内开源社区还需要培养,"不忘初心,砥砺前行" 😄 |
This was referenced Jul 16, 2024
@unicornx 可以帮忙整理出一定的commit,PR的规范吗?如能形成issue,可以钉在置顶上。谢谢 |
好的,我找时间来整理一下 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe problem solved by the proposed feature
Describe your preferred solution
部分解决方案见问题描述
Describe possible alternatives
No response
The text was updated successfully, but these errors were encountered: