前情提要: https://www.v2ex.com/t/543954
第二季: https://www.v2ex.com/t/544369
在前两季播出后,收到了观众朋友们的许多留言,在这里整理回复如下:
闲的蛋疼 /你这也叫开源?
谁能帮我向他解释一下这是不是开源?我不知道该怎么解释。如果修 code style 不算开源的话,我刚刚发现并顺手修复了一个 p3c 插件的 bug,这算开源么? https://github.com/alibaba/p3c/pull/472
你发烧了吧 /你这是杠,不是开源 /你是杠精 /你发一次贴就行了,再发就是蹭热度了 /吃饱了撑的
“发现问题,提交代码解决问题”对你们就这么难以理解么?到今天为止我在 GitHub 上已经提交了 370 个 PR,我就是爱给开源项目提交代码,在你眼里都是杠是不是?你以为谁都跟你一样只会动嘴当喷子?昨天我跟阿里巴巴开源负责人聊过了,人家都没意见,轮到你来唧唧歪歪?称你们一声键盘侠不过分吧?
工作量不饱和
这些 PR 都是早晚非工作时间写的。我公司不加班,仍招人,有兴趣可以投递。如果对我的工作量有意见可以向我的主管投诉。
刚刚向Sentinel 提交了 PR: https://github.com/alibaba/Sentinel/issues/573
利益相关:前阿里巴巴员工,现 Gradle core team developer
1
blindpirate OP |
2
haozhang 2019-03-15 07:39:59 +08:00 via Android
可能他们已经 block 你了,毕竟你把他们脸都扇肿了
|
3
honeycomb 2019-03-15 08:29:04 +08:00 via Android 1
@blindpirate 发现之前就已经屏蔽这三个人了
|
4
pabupa 2019-03-15 08:36:30 +08:00 via Android
您真棒👍,没有错。
前面是我脑袋发烧了,不了解情况,向您道歉。 意思其实很简单,不是说您这样的 pr 不是开源,也不是说您做的不对。只是我的思维有局限吧,,,一直以为修 bug 或者有程序优化的才算…… 但是还是感觉怪怪的,就像是您在帮别人擦脸上的灰。您明白我意思吧……可能我的格局不够,没有把把别人当作我自己。您是前阿里巴巴的员工,当然没有什么问题了。 我绝对不是什么键盘侠,,,,对您也没有什么恶意。 |
5
xiaohuamao 2019-03-15 08:39:41 +08:00 via iPhone
轮循?每天来一遍就没意思了,况且阿里又没有支付薪酬,为什么要花费自己时间精力,帮助阿里改进?
|
6
azh7138m 2019-03-15 08:54:52 +08:00 via Android
emmm,java 都是手动的吗?没有 format 工具吗?
讲道理我也觉得手动改一次两次就好了,可这么多格式问题,不应该用 format 工具自动做掉吗?觉得手动做这些很不程序员。 |
7
huanyues 2019-03-15 08:56:45 +08:00 via iPhone
我觉得挺好的。。。有句话说,圣母(而不是圣母婊)是推动人类进步的最大动力。做事总比坐在那里瞎哔哔要强。那些什么也不做只知道瞎哔哔的人,别人做点事还要阻止的人,可能就是阻止人类进步的圣母婊了。
|
8
maxiaofeng 2019-03-15 09:22:14 +08:00
做了一件你自己觉得很有意义的事, 而别人觉得没有意义的事!! 你也没必要站在道德的制高点。
|
9
gscoder 2019-03-15 09:32:54 +08:00
支持,不懂那些酸的人。搞开源最大的好处不就是提升能力吗
|
10
ericgui 2019-03-15 10:25:27 +08:00
第三次支持楼主
|
11
lusi1990 2019-03-15 10:43:56 +08:00
今天的 V 很火爆啊
|
12
Showfom 2019-03-15 10:57:44 +08:00 via iPhone
@blindpirate 那种傻逼没必要去鸟他们的 直 B 了就完事
|
13
shaohan0228 2019-03-15 11:29:54 +08:00
还是支持的,如果规范出来,无人遵守的话,规范就没有意义了, 这也能看出,阿里提出的规范在内部也并非完全推广,通过这种方式,一方面规范内部代码,另一方面,也会凸显出规范中本身不合理的地方,进一步的去更好的完善规范。
|
14
LanAiFaZuo 2019-03-15 11:31:02 +08:00
坐等
帮助阿里巴巴遵守阿里巴巴规范(第 100 季) |
15
wzxlovesy 2019-03-15 11:39:36 +08:00 via Android
没什么营养
|
16
adminsam 2019-03-15 11:44:39 +08:00
登陆支持,确实是双赢的一件事 👍
|
17
realPipiz 2019-03-15 11:50:21 +08:00 via iPhone
发一个帖子持续更新就行了,别发这么多
|
18
blindpirate OP @realPipiz 不。
|
19
dajj 2019-03-15 12:39:56 +08:00
楼主很棒
|
20
zhix 2019-03-15 12:49:06 +08:00
支持 👍,谁谁谁说中国开源生态差,现在有了先驱者又群起而攻之,大型讽刺现场。
|
21
smeraldo 2019-03-15 13:30:49 +08:00
@SuppressWarnings("PMD.ThreadPoolCreationRule")
没仔细看,不过这也行啊?😂 |
22
lyoe 2019-03-15 13:36:34 +08:00
挺好啊,支持 LZ。认真才有收获。
|
23
yoshiyuki 2019-03-15 16:43:14 +08:00
哪个公司呀
|
24
freedomshi 2019-03-15 19:19:01 +08:00
PP 呢?哈哈哈,还“此人不值得”,哈哈哈哈哈,笑死我了。。。
|
25
ilgharkus 2019-03-15 19:27:35 +08:00
第三次支持楼主。
|
26
ilgharkus 2019-03-15 19:31:24 +08:00
同时庆祝 V2 第一部连续剧开播
|
27
blindpirate OP @xiaohuamao 因为我喜欢写代码呀。
|
28
blindpirate OP @azh7138m 我不知道你用的什么工具,我用 IDEA,在单个文件或者项目上 右键 Analyze - Run inspections by name - 选择修复类型 - Fix 即可。
|
29
blindpirate OP @smeraldo 这样做的目的是保证代码至少从此以后不会变得更差。
|
30
smeraldo 2019-03-15 22:33:05 +08:00
@blindpirate 但..这只是不提示了啊..
|
31
blindpirate OP @smeraldo 我是这么理解的:
首先,对于不能 /不想修复的问题,@SuppressWarnings 是常规操作。在 Sentinel 项目中,说句实话,我没有把握把这几个问题修复好(类似的注解还有 @SuppressFBWarnings,是 FindBugs 引入的,它需要额外的注解是因为 FB 扫描的是字节码,字节码中已经没有 @SuppressWarnings 信息了)。 其次,仅仅 @SuppressWarnings 不会使代码变得更坏——它是编译期注解,对编译后的结果没有任何影响。好处是从此之后新的违背代码规范的问题能在第一时间被发现。 最后,假如哪一天团队想要修复相关问题,全局搜索相应的 @SuppressWarnings 注解即可,无需运行构建。 这让我想起来,gradle 在迁移 Java9 的过程中,发现有很多测试在 Java9 上不能通过。我们当时的解决方案是,引入了一个 FIX_TO_WORK_ON_JAVA9 注解,在 Java9 上直接忽略这些测试: https://github.com/gradle/gradle/blob/69e400f5f00d60684c75827ecf61d3521d333467/subprojects/internal-testing/src/main/groovy/org/gradle/util/TestPrecondition.groovy#L106。忽略这些测试并不意味着从此不管了,而只是代表这些问题我们知道了,当前限于实际无法处理。绝大多数被忽略的 Java9 上的测试已经在去年夏天被修复掉了。 |
32
smeraldo 2019-03-17 19:00:30 +08:00
@blindpirate 用意是好的,但有没有用那得看执行力了,搞不好以后还是没人管。。
|
33
blindpirate OP @smeraldo 从目前和三个 merge 了我的 PR 的团队的接触来看,我对他们三个团队很有信心。但是从跟你的对话来看,我对你的团队没有信心。
|
34
smeraldo 2019-03-18 19:22:02 +08:00 via Android
@blindpirate 😂好巧,我也没有信心
|