![]() | 1 KagurazakaNyaa 145 天前 质量高,结构清晰,文档齐全,侵入性低的代码容易被合并,最好风格和其它维护者保持一致 |
2 renmu 145 天前 via Android ![]() 先提 issue |
3 whileFalse 145 天前 一次解决一个问题更容易接受 |
4 craftsmanship 145 天前 via Android 楼上老哥们说得对 从个人经历来看 遇到好的维护者 他可能会很耐心地告诉你怎么改 所以哪怕水平不是很够 在多次沟通修改之后 PR 也是有机会被接受的 但提 PR 之前一定要先提 issue 跟维护者们沟通好你想改进的点 是不是人家需要的 得到对方的同意之后再动手提 PR 不然很可能你耗费了很多精力和热情 却吃个闭门羹 反过头来还想骂他们不识好歹 |
![]() | 5 MajestySolor 145 天前 ![]() 先提 issue ,充分交流 不提 issue 直接 pr 的懒得看,至少我就是这样 |
![]() | 6 fields 145 天前 一次一个特性呀 清楚的描述这次修改做了什么修改、进行了哪些测试 跟代码量多少没关系吧 |
7 foolishcrab 145 天前 via iPhone 现在基本只有起步阶段的项目的 good first issue 比较容易合了,其他稳定下来的基本都要花非常长的时间融入社区。基本上是一个逐步建立 reputation 的过程,跟上班的代码一样的。 而且这两年开源社区甚至越来越封闭了,挺没意思的 |