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

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

前情提要: https://tanronggui.xyz/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。有任何问题欢迎在此贴中讨论。

7154 次点击
所在节点    Java
59 条回复
dfjslkjdf
2019-03-14 23:53:09 +08:00
@blindpirate
你要是 qa,我收回我的话,我不知道啥叫一个好的 qa。
dfjslkjdf
2019-03-14 23:54:09 +08:00
@David1119
"吃饱了宁可吃鸡、王者、水 v2 也不愿去帮开源项目改个 typo ”
改 typo 这么有意思吗,能超过吃鸡,王者、水 v2 ?
yuikns
2019-03-15 00:23:48 +08:00
勿以恶小而为之,勿以善小而不为。

如楼主所说,改 typo 是向开源社区做贡献的开始,不仅因为它让新手们可以开始学会正确地沟通和协作,也可以让一个很好的项目的 #19 的作者可以将更多精力花费在更多 feature 上。

相反的,作为一个评论者,动不动就论心定罪,恐怕脱袜子来了,都能说人家一句抄 Unix 吧。这也是一种做法。

前者枯燥而收益不大,后者当场站在了世界之巅。当然选后者。
ericgui
2019-03-15 00:36:16 +08:00
支持楼主
zwh2698
2019-03-15 00:44:41 +08:00
楼主,规范已经是公司建立影响行销的手段,你认真了
Cabana
2019-03-15 00:47:41 +08:00
支持楼主,首先不管楼主心态是怎样的,但至少是真的有在「做」,而不是站在 xx 的制高点评头论足
dan666
2019-03-15 00:59:59 +08:00
其实如果有人给我的丑代码提修正规范——而且是按照我自己的规范来提的,我很乐意啊——这不正是我从开源社区获得到的吗
dangyuluo
2019-03-15 04:50:58 +08:00
黑一下得了啊,还得工作呢
lxml
2019-03-15 04:57:55 +08:00
想问楼上的,认真有什么错?
ironMan1995
2019-03-15 05:41:38 +08:00
支持楼主,我强烈建议那些只会嘴上叫嚣,没有一点实际行动的人,闭上你们臭嘴。教育为开源社区做贡献的人,有些人还不配。
TuringGunner
2019-03-15 08:41:16 +08:00
支持楼主,自己出的规范自己不遵守,这不搞笑么
yoshiyuki
2019-03-15 09:42:07 +08:00
实干胜空谈
fatedier
2019-03-15 09:43:26 +08:00
规范代码,提倡大家一起来改进,怎么反而有人嘲讽?
本来就是一件吃力不讨好的事,有点体力活的意思,但是也可以让更多的人接触开源项目,顺带学习一下大厂的规范和经验,对提 PR 的人和项目社区来说是双赢的事情。
mentalkiller
2019-03-15 09:52:07 +08:00
支持楼主。楼主对于开源社区是实实在在的贡献,即使每一步可能都很微小。但是相比于坐在制高点指点江山的键盘侠,真的是高得不知道哪里去了。
Raymon111111
2019-03-15 10:22:03 +08:00
@ily433664 并不是, 阿里那个规范里面很多项都是标注"强制", 如果认为标注"强制"的可以不用改, 那么规范本身应该把"强制"给降级成"推荐". 这也不是推进规范更好的发展吗?
Shintaku
2019-03-15 10:58:05 +08:00
@dfjslkjdf 感觉还是有必要的,我司这里要不加大括号都过不了规范检查
Nostalgiaaaa
2019-03-15 11:03:43 +08:00
能实干的人已经很佩服了,支持!
ily433664
2019-03-15 11:25:34 +08:00
@Raymon111111 规范里的“强制”是指,你遵守规范的前提下,请搞清楚前后的因果关系
Raymon111111
2019-03-15 11:48:51 +08:00
@ily433664 阿里内部出了个规范, 里面有"强制", 说明这些要求都是不遵守会造成重大影响的. 但是阿里内部的团队还可以不去遵守, 说明这个"强制"并不合理, 应该被修改.

不玩文字游戏了.

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

https://tanronggui.xyz/t/544369

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

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

© 2021 V2EX