帮助阿里巴巴遵守阿里巴巴规范(第二季)

2019-03-14 06:39:27 +08:00
 blindpirate

前情提要: https://www.v2ex.com/t/543954

想要为开源项目做贡献,但是苦于找不到门路?请加入我,一起向阿里巴巴提交 PR,修正阿里巴巴的开源项目中违背 https://github.com/alibaba/p3c 规范的代码吧!许多人向开源社区做贡献就是 fix typo/code style 开始的,我也不例外。现在有一个好机会向知名项目提交代码,并且十分容易!

方法参见: https://github.com/blindpirate/does-alibaba-follow-their-own-guidelines/blob/master/README.md#%E5%B8%AE%E5%8A%A9%E9%98%BF%E9%87%8C%E5%B7%B4%E5%B7%B4%E9%81%B5%E5%AE%88%E9%98%BF%E9%87%8C%E5%B7%B4%E5%B7%B4%E8%A7%84%E8%8C%83

  <plugin>
    <groupId>org.apache.maven.plugins</groupId>
    <artifactId>maven-pmd-plugin</artifactId>
    <version>3.8</version>
    <configuration>
      <sourceEncoding>${project.build.sourceEncoding}</sourceEncoding>
      <targetJdk>1.8</targetJdk>
      <printFailingErrors>true</printFailingErrors>
      <rulesets>
        <ruleset>rulesets/java/ali-comment.xml</ruleset>
        <ruleset>rulesets/java/ali-concurrent.xml</ruleset>
        <ruleset>rulesets/java/ali-constant.xml</ruleset>
        <ruleset>rulesets/java/ali-exception.xml</ruleset>
        <ruleset>rulesets/java/ali-flowcontrol.xml</ruleset>
        <ruleset>rulesets/java/ali-naming.xml</ruleset>
        <ruleset>rulesets/java/ali-oop.xml</ruleset>
        <ruleset>rulesets/java/ali-orm.xml</ruleset>
        <ruleset>rulesets/java/ali-other.xml</ruleset>
        <ruleset>rulesets/java/ali-set.xml</ruleset>
      </rulesets>
    </configuration>
    <executions>
      <execution>
        <phase>verify</phase>
        <goals>
          <goal>check</goal>
        </goals>
      </execution>
    </executions>
    <dependencies>
      <dependency>
        <groupId>com.alibaba.p3c</groupId>
        <artifactId>p3c-pmd</artifactId>
        <version>1.3.6</version>
      </dependency>
    </dependencies>
  </plugin>

非常简单,唯一困难的地方就是迈出第一步。我向 arthas 项目提交了一个 PR 作为样例: https://github.com/alibaba/arthas/pull/568

可以看到几乎都是体力活,十五分钟足够了。

欢迎 review 我的 PR。有任何问题欢迎在此贴中讨论。

6874 次点击
所在节点    Java
59 条回复
blindpirate
2019-03-14 06:40:09 +08:00
@PP 我已经提交了两个 PR 了,你呢?你还在那里高谈阔论么?
pabupa
2019-03-14 07:39:05 +08:00
你发高烧了吧…………
blindpirate
2019-03-14 07:51:08 +08:00
@pabupa 我刚数了一下我迄今为止已经在 GitHub 上提交了 365 个 PR 了,可能发了 365 次高烧吧。
mswlander
2019-03-14 07:51:59 +08:00
工作量不够饱和。。。
skd918
2019-03-14 07:57:55 +08:00
你气量可真够可以的
KinneyZhang
2019-03-14 08:00:47 +08:00
意思是阿里自己的项目代码没有遵守自己的规范?有点滑稽^o^
PALELESS
2019-03-14 08:12:18 +08:00
这不算为开源做贡献,这算杠,写代码的人八成很感谢你来修人家的代码
fzy0728
2019-03-14 09:01:04 +08:00
工作量不饱和...
hirasawayui
2019-03-14 09:03:12 +08:00
这也算开源?
honeycomb
2019-03-14 09:03:48 +08:00
不错
hhhsuan
2019-03-14 09:05:40 +08:00
支持,阿里的规范自己都不遵守,拿出来忽悠人
jeffersonpig
2019-03-14 09:11:33 +08:00
阿里付我工资了吗我就要帮它?总觉得是阿里的人自己推规范推不动,想利用外面的人来施加压力,大家千万别上当受骗被利用做了免费苦力
VoidChen
2019-03-14 09:35:02 +08:00
BLOCK,感觉 V2 也快呆不下去了,一楼还 @PP,还删了假装什么都没发生= =
binux
2019-03-14 09:45:22 +08:00
这很正常,这在开源中很正常,无论是在国内还是国外都很正常。
当年那个给我项目提 pep8 的,人家现在已经是 CTO 了。
yhxx
2019-03-14 09:50:21 +08:00
@hirasawayui 这都不算,难道只有 “致阿里巴巴,此人不值得”,这样才算?
morethansean
2019-03-14 09:51:08 +08:00
@VoidChen 哈哈哈,看原文我觉得 PP 言辞和善观点也挺正的,不知道楼主为啥这么杠……
mingqing
2019-03-14 09:51:10 +08:00
规范是经历过大量基础实践才总结出来的,原先的实践也必须通过一定的时间去迭代改进,不可能一出规范所有的项目都一定遵守(主要是给新项目做标准)
huangdayu
2019-03-14 09:51:12 +08:00
声援
VoidChen
2019-03-14 10:03:04 +08:00
@morethansean 大概是“除了我以外在座各位都是键盘侠,别瞎 BB 打扰到我用 TNT 提 PR 了”。其实事情本身是件好事,想要改变整个环境规范也非一日之功,就是心急了想提前摘果子= =
leeloc
2019-03-14 11:53:46 +08:00
支持。看看这些冷嘲热讽的,还言辞和善观点正

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/544369

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX