V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member Sign In
• 请不要在回答技术问题时复制粘贴 AI 生成的内容
我在 GitHub 上给个项目提交了 PR，就两行代码，非常简单，然后他们几个 Committer 跟我已经“撕”了很久了，貌似看不到 Merge 的希望了……
int64ago · int64ago · Dec 31, 2015 · 10992 viewsThis topic created in 3772 days ago, the information mentioned may be changed or developed.
唉，他们太固执了……
刚刚又放了两个截图，我确信这是很简单得问题，也是很简单逻辑
难道一直是我表达问题？
 |
1
fuge Dec 31, 2015
好欢乐的样子
|
 |
3
yzl0208994 Dec 31, 2015
这个功能我觉得会让人不爽
|
 |
4
bdbai Dec 31, 2015 via iPhone
个人觉得利用浏览器缓存是比较 hack 的做法，确实有"浪费宽带"的嫌疑。
|
 |
5
int64ago OP @yzl0208994 为什么呢？
|
|
7
yzl0208994 Dec 31, 2015
@int64ago 不经用户同意乱动作，有种被侵犯的感觉。如果打开会慢，我也会接受的；但是不喜欢别人做决定。
|
 |
8
saber000 Dec 31, 2015
邀请楼主回答知乎问题:
https://www.zhihu.com/question/38756228 |
 |
9
luoway Dec 31, 2015  1
自从当了程序员，最常听到的就是程序员说用户体验了。
|
 |
10
11 Dec 31, 2015 1
这个功能我感觉有点牵强了。一个好的功能应该是符合用户预期的，这里用户点击了 copy ，但是会发出一个 get 请求，这是不符合用户预期（直觉）的。
|
|
11
int64ago OP @11 这我是不同意的，比如 Google Instant predictions 我觉得就很好
当然你也可以说 Google Instant predictions 可以自己设置关掉，不过很多时候如果你的行为 A 接下来基本是行为 B 的时候，提前为行为 B 做一些准备好事 当然，这所有的前提是不能作恶 |
 |
12
zcbenz Dec 31, 2015 2
大部分 maintainer 对于增加很少人会用到的功能是非常抵触的，
|
 |
13
nbndco Dec 31, 2015 1
这个需求毫无道理，如果是我应该会直接 close 掉。
|
 |
14
gamexg Dec 31, 2015
相关协议升级，库在新协议下直接挂了，提交了个 PR ，作者回复感谢，但是很长时间也不合并的路过...
|
 |
15
bk201 Dec 31, 2015 via iPhone
如果能自定义是否启用就好了，你这样做只是满足一小部分人的需求。
|
|
16
int64ago OP @bk201 其实我承认这也是自己需求，但是我不太乐意自己搭一个（虽然很简单），如果官方不 Merge 我是理解的，但是里面我需要把我的想法阐述清楚，也因为官方认真讨论而感觉不错
|
 |
17
crs0910 Dec 31, 2015 via iPhone
有没有可能怕被人说造假
|
 |
18
vanxining Dec 31, 2015 via Android
那个人似乎并没有动摇。只是一种婉拒，我认为。
话说我并不知道楼主和那几个人在讨论什么，隔行如隔山啊~ |
|
19
vanxining Dec 31, 2015 via Android
每次我想说不的时候，我都是跟别人说，可以再讨论讨论……
|
 |
20
marffin Dec 31, 2015
merge 不了很正常，做之前要跟人家沟通好才好，否则就会热脸贴到冷屁股上。试想有个人冷不丁的给你的项目发个 PR ，你会是什么感觉。
|
 |
21
SmiteChow Dec 31, 2015
这个 PR 放我也不会合的，那两行代码不应该出现在那个地方。
|
 |
22
BOYPT Dec 31, 2015
如果是我就直接 close 掉 +1
|
 |
23
skydiver Dec 31, 2015 via iPad
感觉像是作弊。。
|
 |
24
xavierchow Jan 1, 2016
建议先提 issue 再发 PR ，谋定而后动。
|
 |
25
demoslam Jan 1, 2016
为什么 i 不大写 I 。。。
|
 |
26
yangqi Jan 1, 2016
同意这个不是很妥，出发点是好的，但是用户预期的是点了复制就是复制，你执行别的至少要给用户选择的权利吧? 比如加一个选项之类的，光是这两行代码感觉不妥
|
 |
27
imn1 Jan 1, 2016
copy 下一步一定是用么？
|
 |
28
Perry Jan 1, 2016
agree to disagree
|
 |
29
7gugu Jan 1, 2016
感觉这个功能是挺好的,但必须加上一个开关才好,假如用户的带宽非常差,可能会造成资源浪费,导致用户体验变差
|
 |
30
sxd Jan 1, 2016
站在不同角度看待问题 考虑就会不一样
|
 |
31
v1024 Jan 1, 2016 via iPhone
close+1 没什么可讨论的
|
 |
32
latyas Jan 1, 2016
你没发现你在和一个中国人用英文讨论么？
|
 |
33
spacewander Jan 1, 2016 via Android
锦上添花的 pr 能不能合并基本看运气。
|
 |
35
wuchizhitu1988 Jan 1, 2016 via iPhone
我觉得也不妥 基本是从自己需求出发考虑的
|
 |
36
holulu Jan 2, 2016
项目管理对于增加的功能有考量不是应该的么？这不是固不固执的问题，而且是就事论事啊。其实如果这只是你的需求，你完全可以维护自己的版本，不一定要 PR 。
|
Select Language