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