是我不懂礼貌太骄傲了吗?有点懵逼

67 天前
 kingmo888

请问,关闭别人的 PR 是太骄傲的行为吗?

早前分享过一个 rustdesk 的中间件小工具不知道大家是否有印象:

rustdesk-api-server

很少主持开源项目,顶多是有的项目给别人提 PR (提完了我很少主动关注,这是我的不对。)

最近有个网友提了 2 个 pr ,其中一个因为功能重复且不太符合我对项目的设计,讨论后我给关了,然后对方关了另外个 pr ,说我太骄傲,我不太理解。

#129

在此请教下开源礼节。谢谢大家。

3029 次点击
所在节点    问与答
16 条回复
coderluan
67 天前
讲道理啊,对方是认为你"也没有讨论这个功能是否合适,就直接 close 这个 PR",这和“关闭别人的 PR"完全是两回事,前者肯定不好,后者肯定没问题,至于楼主是前者还是后者,我就不判断了,毕竟这涉及项目功能。

PS:我认为此场景的下正确的用词是傲慢,不是骄傲。
dawn009
67 天前
你没问题。
只提一个可以改进的地方:关的时候同时附带上简短的解释。
zizon
67 天前
题外话,这个 pr 还动了代码格式.
whoosy
67 天前
爱用不用
qianxu2001
67 天前
看到 issue 置顶就难绷
cmdOptionKana
67 天前
你不够傲慢,而且也太介意傲慢了。中国文化过度宣传谦虚,其实你是可以傲慢的。

当你真的傲慢了,他们反而不说你了(一是因为看你不好惹,二是因为这招已经破不了你的防,浪费力气)
tool2dx
67 天前
看了一眼 pr 合并的比例,挺高的,说明 OP 完全没问题。

对比一下 ffmpeg ,合并比例才 30%不到,那才叫惨。
kingmo888
67 天前
感谢各位朋友。
cshever999
67 天前
这有什么礼貌不礼貌,你用洋文回复,你看他还说不说。不符合你项目设计逻辑,那就是不合适,我觉得你态度已经表明的很清楚了,没问题。
yunshangzhou
67 天前
你是 CR 主导者,当然有权关闭了。而且退一步讲,对方就算愿意听你的建议修改代码,其实算双方都二刷了时间成本,没必要,该关关
zeusho871
67 天前
无 star 不准提 issues
a1b2c3T
67 天前
@qianxu2001 #5 我还懵逼了一下,哈哈哈哈,直到点进项目看到 issue 然后回来给你点赞
imnpc
67 天前
我也用的这个 功能很满意 除了 web 控制 从没成功过
kingmo888
67 天前
kingmo888
67 天前
@imnpc web 控制只支持 http 下控制
kingmo888
67 天前
再次谢谢大家回复

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/1069845

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX