我在 GitHub 上给个项目提交了 PR，就两行代码，非常简单，然后他们几个 Committer 跟我已经“撕”了很久了，貌似看不到 Merge 的希望了……

2015-12-31 16:57:21 +08:00
 int64ago

唉，他们太固执了……
刚刚又放了两个截图，我确信这是很简单得问题，也是很简单逻辑
难道一直是我表达问题？

https://github.com/cdnjs/new-website/pull/106

10376 次点击
所在节点    程序员
36 条回复
SmiteChow
2015-12-31 22:25:41 +08:00
这个 PR 放我也不会合的，那两行代码不应该出现在那个地方。
BOYPT
2015-12-31 22:29:15 +08:00
如果是我就直接 close 掉 +1
skydiver
2015-12-31 22:31:41 +08:00
感觉像是作弊。。
xavierchow
2016-01-01 00:07:41 +08:00
建议先提 issue 再发 PR ，谋定而后动。
demoslam
2016-01-01 01:33:03 +08:00
为什么 i 不大写 I 。。。
yangqi
2016-01-01 05:35:14 +08:00
同意这个不是很妥，出发点是好的，但是用户预期的是点了复制就是复制，你执行别的至少要给用户选择的权利吧? 比如加一个选项之类的，光是这两行代码感觉不妥
imn1
2016-01-01 05:38:23 +08:00
copy 下一步一定是用么？
Perry
2016-01-01 05:47:03 +08:00
agree to disagree
7gugu
2016-01-01 11:05:33 +08:00
感觉这个功能是挺好的,但必须加上一个开关才好,假如用户的带宽非常差,可能会造成资源浪费,导致用户体验变差
sxd
2016-01-01 14:55:45 +08:00
站在不同角度看待问题 考虑就会不一样
v1024
2016-01-01 15:29:34 +08:00
close+1 没什么可讨论的
latyas
2016-01-01 16:37:57 +08:00
你没发现你在和一个中国人用英文讨论么？
spacewander
2016-01-01 16:50:27 +08:00
锦上添花的 pr 能不能合并基本看运气。
int64ago
2016-01-01 16:56:07 +08:00
@latyas 他是台湾人
wuchizhitu1988
2016-01-01 19:22:28 +08:00
我觉得也不妥 基本是从自己需求出发考虑的
holulu
2016-01-02 17:26:51 +08:00
项目管理对于增加的功能有考量不是应该的么？这不是固不固执的问题，而且是就事论事啊。其实如果这只是你的需求，你完全可以维护自己的版本，不一定要 PR 。

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

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

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

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

© 2021 V2EX