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