https://github.com/ant-design/ant-design/issues/13098#issuecomment-449754024
做框架不好好修 bug,掺 x 为什么这么积极?
问了一圈,我认识的前辈里 9 月 19 号就已经发现了这段“菜( di )单( lei )”
https://github.com/ant-design/ant-design/commit/00aebeb9756afecc884ad48486084836b9a2707a
始作俑者甚至还表示:changelog 里无须体现( https://github.com/ant-design/ant-design/commit/00aebeb9756afecc884ad48486084836b9a2707a#commitcomment-30445378 )
现在爆发出来我觉得非常正常,因为开发者既然用你这个库就是信你这块牌子,很少人一个个文件审查、一个个提交的看。
各位怎么看?我是觉得非常不合适。
https://github.com/ant-design/ant-design/issues/13818#issuecomment-449753026
怕不是只要做好坐牢的准备,什么恶意代码都敢掺咯?
这里想多说两句。
首先,Ant Design 对于前端业界的贡献是很大的,虽然这次出现了这样的人为疏失,但不得不说这个项目已经是为数不多完整可用、并且不断在迭代的项目了。我相信大多数人就算在使用中遇到这样那样的问题,最终还是会感谢这个项目的开发人员做出的贡献的。
那问题出在哪儿?你不能要求所有开发者都逐行审查自己使用的库的代码,因为这在一半工程实践里不现实。你也不能要求开源项目的作者付出120% 的心力来完善项目保证没有问题,因为这也不现实。
怪谁呢?想来想去,最后发现还是国人的开源模式和做法上的问题。比如我在主楼提到的,已经有大佬提早在审查代码的时候发现了,但没有提交pr 和issue,只是在自己的项目里进行了相关的修正,最终没有把问题扼杀在摇篮里。问起大佬为什么不提交?大佬的回答:“反正这不是我的库,干什么多管闲事?”。联想起到现在还有很多朋友发提问贴,得到回复并解决之后就编辑掉帖子的做法,突然就感觉很无力。
是啊,这就是现实。
希望开发者不要再进行独断专横的操作了,您在开发的代码被很多前端项目依赖,请不要辜负使用者对您的信任。
希望各位开发者发现问题或者解决问题的时候,请不要吝啬分享,说不定你做出的贡献可以让很多人收益。
在使用各种服务的各种功能前,请务必摸清其中规则。比如:github 的issue 并不是用来骂街和观光的,npm 不是用来传乱七八糟东西的。
你索取是你的权力,但别人提供并不是义务。
生活不易,愿国内开源氛围能够越来越好吧。
![]() | 1 SCaffrey 2018-12-25 10:05:32 +08:00 via Android |
2 Justin13 2018-12-25 10:07:24 +08:00 via Android 正常,官方教程里面的 datepicker 导入方法,有一个完全导不进来。绝了,我他妈那天加班好久翻代码才发现得从 es 文件夹里导。 |
![]() | 3 jianzhihao1996 2018-12-25 10:08:03 +08:00 看到被炒的老哥莫名心痛 |
![]() | 4 hdr01 2018-12-25 10:11:07 +08:00 ![]() console 里玩彩蛋就行了,这种行为就是在代码里下毒! |
![]() | 5 qhxin 2018-12-25 10:12:33 +08:00 哈哈,今天看到了 ![]() |
![]() | 6 Vegetable 2018-12-25 10:14:08 +08:00 输出爆炸 |
![]() | 7 ericgui 2018-12-25 10:14:22 +08:00 @jianzhihao1996 什么情况? |
![]() | 9 fe619742721 2018-12-25 10:16:16 +08:00 天秀操作 |
![]() | 10 yiqiao 2018-12-25 10:18:05 +08:00 ![]() |
![]() | 11 hlwjia PRO ![]() issue 里面有条 comment 很值得思考 "I really want to use Ant for future projects but decisions like this concern me." 先不说修不修 bug 的问题,但是这个彩蛋作为一个默认出现、没有设置可以配置、需要用 !important 的方式来 overwrite 的决定,是怎么通过需求审核、然后代码审核、然后再合并的? Ant Design 的团队应该意识到你们的产品已经不是小打小闹了,要有国际水准。 |
![]() | 12 airyland 2018-12-25 10:20:55 +08:00 via iPhone ![]() 不支持,不理解,所谓的彩蛋对别人就是事故。 |
![]() | 13 maichael 2018-12-25 10:22:05 +08:00 愚蠢到没话说。 |
![]() | 14 zhuoyan 2018-12-25 10:22:37 +08:00 秀啊 |
![]() | 16 oott123 2018-12-25 10:25:03 +08:00 “早就做好了挨骂的准备” 珍爱生命,拒绝 ant design,毕竟你可没有做好挨骂的准备。 |
![]() | 17 hitfm 2018-12-25 10:28:59 +08:00 修复方案也是牛逼啊,直接样式 override 掉~ |
18 guoshencheng1 2018-12-25 10:30:30 +08:00 在用开源框架之前没有自己看过源码,在重要的项目里使用这样的框架,也要考虑一下自己的问题吧 |
![]() | 19 wsstest 2018-12-25 10:31:49 +08:00 心疼那些被这个菜单祸害的开发们 |
20 shylockhg 2018-12-25 10:34:02 +08:00 @jianzhihao1996 真的是因为这个? |
![]() | 21 WuwuGin 2018-12-25 10:34:11 +08:00 @guoshencheng1 这种根本不属于框架的范畴了,框架开箱即用也是特点,况且没有文档说明,你是看了多少框架的所有源码? |
![]() | 22 Enivel 2018-12-25 10:35:44 +08:00 ![]() @guoshencheng1 专业! 就应该像这位老哥一样, 通读项目源码. 不然出了 bug 就是你们自己的问题. 另外 chromium 源码看到哪了? |
![]() | 23 FunnyCodingXu 2018-12-25 10:35:53 +08:00 ![]() @guoshencheng1 老哥前端后端? 所有库的源码你都看过? |
24 maemual 2018-12-25 10:37:02 +08:00 @guoshencheng1 #18 如果用的所有开源工具全都去看一遍源码,那每天还干不干活了。。。。 |
![]() | 25 zpf124 2018-12-25 10:37:10 +08:00 从逻辑学的角度讲,changelog 里写出来了还能叫做彩蛋么... 不过作为一个给开发者提供的工具,里面加这种东西与其叫菜单不如叫下毒。 |
![]() | 26 SakuraKuma 2018-12-25 10:39:24 +08:00 这操作。。 要是对企业,怕是打包走人了。。 某些企业对这些西方节日,十分。。emmm |
![]() | 27 UIXX 2018-12-25 10:40:58 +08:00 很影响 Ant Design 口碑的一次事件,技术的问题可以改,但是信任出现裂缝这是很难修补的。 同时充分暴露出了这个团队的缺点:不够专业、不够负责,没有服务意识等。 一个库开源出来,为一定数量的人造成了影响、也为自己收获了口碑,那这个库作者势必要为自己库代码(尤其是私货)给别人造成麻烦负责的,但“无须体现”让我觉得 Ant Design 掉进了一颗老鼠屎,以后会尽量避免使用这种团队开发的产品。 |
![]() | 28 popu111 2018-12-25 10:41:34 +08:00 @guoshencheng1 即使看完了,也顶不住半路喂屎啊……你还每天追着 commit 看不成 |
![]() | 29 wengjin456123 2018-12-25 10:43:32 +08:00 我司也使用 antdesgin,准备慢慢不用了,寻找替代 |
![]() | 30 xiaochocking 2018-12-25 10:44:01 +08:00 @popu111 需要更新版本的时候再看一遍 ![]() |
![]() | 31 leisure 2018-12-25 10:45:58 +08:00 我们项目还好还没上线.... |
32 guoshencheng1 2018-12-25 10:46:16 +08:00 ![]() @popu111 对的,我的意思不是 antd 全无责任,身为开发者的我们对自己的应用足够负责就好了,antd 的源码非常简单易懂,简单看一下不成问题,UI 框架这种东西,用了就很少会更新到最新的,追着 commit 其实没什么必要,如果在使用这种简单易懂的开源框架没有看过源码只是跟风的话,这是对自己的应用的不负责。 |
![]() | 33 xAI 2018-12-25 10:47:39 +08:00 via iPhone antd 的开发者对使用者都是爱用不用的态度,感觉只是一个内部项目开源出来了,需求都是优先看内部。根本不是一个开源了让社区能参与的项目。 |
![]() | 34 matsuijurina 2018-12-25 10:48:31 +08:00 via Android ![]() 如果面对的客户所在的国家和区域宗教信仰不同,这样的彩蛋是完全可能导致公司失去合同,赔偿损失,甚至前端下岗的。有些程序员对于维护这种挂着厂牌的开源项目毫无责任感和敬畏感,始终就是一个玩耍的态度,今天塞个彩蛋代码,明天缺钱了再塞一段挖矿代码,不知道是在砸谁的锅。 |
![]() | 35 DEEPPANDA 2018-12-25 10:49:31 +08:00 via iPhone ![]() 人家辛辛苦苦开发良心产品开源给大家免费用,对于那些喷子我只想说:爱用就用,不用滚。 |
![]() | 36 hitfm 2018-12-25 10:50:28 +08:00 @UIXX 同意,态度很重要。定位为企业级框架,出了事情直接抛一个样式覆盖的方案然后关掉 issue 也太随便了。 刚更新了版本去掉了这彩蛋了: https://github.com/ant-design/ant-design/commit/6550df34b639ab0b3bf2c1cbf9b9828735c1fd41 |
37 whileFalse 2018-12-25 10:50:29 +08:00 谁能告我彩蛋的效果是啥 |
38 e9e499d78f 2018-12-25 10:51:34 +08:00 via iPhone ![]() 刚觉是射了一坨在 button 上 |
![]() | 39 Persephone OP |
41 Mariano 2018-12-25 10:54:56 +08:00 |
42 x66 2018-12-25 10:55:02 +08:00 |
43 cyspy 2018-12-25 10:59:11 +08:00 重要的是这个彩蛋真的很丑,而且大部分人看到这东西的第一反应并不能联想到圣诞节,不然也能少些负面反馈 |
![]() | 44 JoyNeop 2018-12-25 11:00:15 +08:00 |
![]() | 45 JoyNeop 2018-12-25 11:00:27 +08:00 搞事搞事 23333 |
![]() | 46 learnshare 2018-12-25 11:00:33 +08:00 所以只推荐 Bootstrap |
![]() | 47 tangler 2018-12-25 11:01:53 +08:00 |
![]() | 48 tangler 2018-12-25 11:02:16 +08:00 |
![]() | 49 ayase252 2018-12-25 11:04:59 +08:00 via iPhone 之前好像看到过这个彩蛋的讨论,可惜关注度不高。现在果然爆炸了。埋彩蛋可以,但是要有控制手段,要有提前告知。我们大多数时候不是在为自己写程序啊。 |
![]() | 50 Timgle168 2018-12-25 11:05:57 +08:00 坑惨了 |
![]() | 51 mritd 2018-12-25 11:06:28 +08:00 ![]() 这波操作是真的秀,很早就说: 珍爱生命,远离阿里开源 |
![]() | 52 mritd 2018-12-25 11:07:19 +08:00 感觉阿里开源面临的不是技术问题,而是信任危机 |
53 foolyf 2018-12-25 11:08:09 +08:00 阿里开源的东西果然一如既往 |
55 JLNR 2018-12-25 11:10:39 +08:00 ![]() 阿里的价值观还是一如既往的秀得我头晕 |
![]() | 56 tabris17 2018-12-25 11:12:00 +08:00 受害者持续增加…… |
![]() | 57 ChristopherWu 2018-12-25 11:12:01 +08:00 @JoyNeop 厉害。。大佬的英语,作为魔法世家的弟子,实在佩服。。 |
![]() | 58 SEARCHINGFREE 2018-12-25 11:12:28 +08:00 前端娱乐圈,又添新乐子 |
59 PhpBestRubbish 2018-12-25 11:12:35 +08:00 真的很讨厌这种东西,还有那种举动给用户头像加上圣诞帽的,不是所有人喜欢 |
![]() | 60 wrq 2018-12-25 11:16:58 +08:00 笑得我肚子疼 |
![]() | ![]() |
![]() | 62 ballshapesdsd 2018-12-25 11:20:28 +08:00 谢罪警告 |
![]() | 63 zpf124 2018-12-25 11:20:34 +08:00 |
![]() | 64 chiou 2018-12-25 11:21:22 +08:00 秀啊,天秀 |
![]() | 65 dragonszy 2018-12-25 11:22:26 +08:00 咋一看以为是 ant-design-vue,没想到是 ant-design 官方,这真的是大事件了。 |
![]() | 66 plqws 2018-12-25 11:24:16 +08:00 @guoshencheng1 我吃饭吃到屎,首先得怪我自己没有把饭拿去做色谱分析? |
67 kuxuan 2018-12-25 11:27:24 +08:00 我用得最多的,赶快修复。 |
![]() | 68 ChristopherWu 2018-12-25 11:30:45 +08:00 ![]() http://qianduanyuleclub/ 前端娱乐圈已更新。 果然永远不会让我失望。。。 |
![]() | 69 OMGZui 2018-12-25 11:31:39 +08:00 不知道怎么评价,请加个春节彩蛋 |
![]() | 70 undeflife 2018-12-25 11:33:02 +08:00 Ant Design An enterprise-class UI design language and React implementation. 这个 enterprise-class UI 是想说 企业级 UI 吗? 有这么个说法? |
![]() | 71 duan602728596 2018-12-25 11:33:34 +08:00 via iPhone 该用还得用。不想用的,要么自己拿 react-components 封装,要么换 vue 去。 |
![]() | 72 jin5354 2018-12-25 11:35:54 +08:00 ![]() @guoshencheng1 可拉倒吧,这种偷摸塞几行的代码,你不逐行扫源码过去根本发现不了,这跟简不简单没关系,只要代码量大花时间一定不会少,打开你项目的 node_modules 文件夹,告诉我里面代码你审计过多少?你能对你项目负多少责? |
73 littlebaozi 2018-12-25 11:35:59 +08:00 脑子瓦特了 |
![]() | 74 aino 2018-12-25 11:37:58 +08:00 ![]() @jianzhihao1996 #3 张谱,前端一枚,老家宁波东钱湖,现在杭州滨江某公司就职。 因为这个彩蛋被裁 |
![]() | 75 zlhsvc 2018-12-25 11:38:28 +08:00 今天可以给你加个彩蛋,明天就能给你来个后门。 |
76 designer 2018-12-25 11:40:51 +08:00 via iPhone ![]() “ hello, 大家好,我是一个在伊朗工作的中国前端,因为使用 antd 进行平台开发,在伊朗官方网站上显示出来圣诞快乐,目前已经被抓,求大家救救 me ” |
![]() | 77 lincanbin 2018-12-25 11:41:00 +08:00 @guoshencheng1 js 的特性,就是自带的库里,什么都没有,一般一个项目 npm install 后就是几百兆的代码。 连个简单的时间格式化都要引用一个库的。 |
![]() | 79 OMGZui 2018-12-25 11:42:51 +08:00 ![]() |
![]() | 80 ericgui 2018-12-25 11:45:15 +08:00 这代码有问题啊,就算是判断圣诞,也应该是 12 月 25 日啊 |
![]() | 81 xAI 2018-12-25 11:46:22 +08:00 看了项目正式版里面还是用的老版本,不受这个影响,但是测试项目里面是最近更新的 antd,有受到影响,不过影响小点。 一个有各国不同宗教人使用的项目,要是真受到这个彩蛋影响,问题还是蛮大的。 后续看能不能找项目替代 antd,UI 框架做这种事问题太严重了。 |
![]() | 82 SEARCHINGFREE 2018-12-25 11:46:30 +08:00 @ericgui 你一定不是前端 |
83 morethansean 2018-12-25 11:46:43 +08:00 via Android @ericgui 一看就不是前端... |
![]() | 84 nezumi85757 2018-12-25 11:49:48 +08:00 等等看官方怎么说 |
85 dbpe 2018-12-25 11:50:06 +08:00 >>Issue 是用来提问题跟解决问题,而不是用来炒热点的,讨论到这份上只能锁了 这个天秀 PS:我感觉是不是内部有矛盾啊.... |
![]() | 86 ChristopherWu 2018-12-25 11:50:15 +08:00 @OMGZui 有点乱.. |
88 civet 2018-12-25 11:55:44 +08:00 |
![]() | 89 Trim21 2018-12-25 11:57:47 +08:00 |
90 ylsc633 2018-12-25 12:00:32 +08:00 @PhpBestRubbish 哈哈 你指的是 掘金吗?? 哈哈哈 最近我发现 这网站给我头像上强行加了个帽子 |
![]() | 91 lincanbin 2018-12-25 12:04:56 +08:00 ![]() ![]() 支付宝 2016 年儿童节的彩蛋,是把所有用户的用户名强制改成“**宝宝”。 可见恶心人的彩蛋是阿里一贯的传统,十分符合阿里价值观,应该给埋这个彩蛋的同学升职加薪。 |
![]() | 92 lincanbin 2018-12-25 12:08:06 +08:00 ![]() ![]() |
![]() | 93 thcode 2018-12-25 12:13:24 +08:00 ![]() 建议 Ant Design 加入愚人节彩蛋,4 月 1 日当天点击按钮随机出现“ you're fooled!”的弹窗 |
95 boris1993 2018-12-25 12:27:53 +08:00 via Android @thcode #92 按照这次的尿性,得写成 Hey~ You are FOOLED! ASSHOLE! LOL! |
96 mercury233 2018-12-25 12:28:00 +08:00 话说回来,哪天国外开源前端库学 notepad++搞个事情怎么办 |
![]() | 99 TingHaiJamiE 2018-12-25 12:33:18 +08:00 ![]() 只有圣诞节有,而春节没有,是突显国际化么( MSL 地区 /国家咋办...)。2333333 |
![]() | 100 miyuki 2018-12-25 12:38:11 +08:00 antd 牛逼 |