• 请不要在回答技术问题时复制粘贴 AI 生成的内容
int64ago
V2EX  ›  程序员

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

  •  
  •   int64ago ·
    int64ago · Dec 31, 2015 · 11471 views
    This topic created in 3966 days ago, the information mentioned may be changed or developed.

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

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

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