Because there isn't an checkstyle configure file of Ali way, 
 So it is impossible to check code style in with GitHub Action. 
 But I strongly suggest use IDE plugin to check your code 
 locally . 





------------------ ???????? ------------------
??????:&nbsp;"Jave-Chen"<kejia.c...@foxmail.com&gt;;
????????:&nbsp;2020??2??8??(??????) ????9:05
??????:&nbsp;"dev"<dev@dolphinscheduler.apache.org&gt;;

????:&nbsp;?????? [CI][Code style] Need help with checking code style check by 
Ali way



The file 
<https://github.com/zhnlk/ali-checkstyle-xml/blob/master/ali-checkstyle.xml&amp;gt;
 is created at 2017, are u sure?




------------------&amp;nbsp;????????&amp;nbsp;------------------
??????:&amp;nbsp;"lidong dai"<dailidon...@gmail.com&amp;gt;;
????????:&amp;nbsp;2020??2??8??(??????) ????8:59
??????:&amp;nbsp;"dev"<dev@dolphinscheduler.apache.org&amp;gt;;
????:&amp;nbsp;"wangyang.03"<wangyang...@bytedance.com&amp;gt;;
????:&amp;nbsp;Re: [CI][Code style] Need help with checking code style check by 
Ali way



hi ,
I see checkstyle.xml (
https://github.com/zhnlk/ali-checkstyle-xml/blob/master/ali-checkstyle.xml)
from
http://mail-archives.apache.org/mod_mbox/dolphinscheduler-dev/202001.mbox/%3Ctencent_6DF620A15E4A57C43A349736CBCD15454907%40qq.com%3E,
is this checkstyle OK ?


<https://github.com/zhnlk/ali-checkstyle-xml/blob/master/ali-checkstyle.xml&amp;gt;


Best Regards
---------------
DolphinScheduler(Incubator) PPMC
Lidong Dai ??????
dailidon...@gmail.com
---------------


Jave-Chen <kejia.c...@foxmail.com&amp;gt; ??2020??2??8?????? ????4:41??????

&amp;gt; Hi, all
&amp;gt; As we discussed about checkstyle, we agreed use Ali code style.
&amp;gt; <
&amp;gt; 
http://mail-archives.apache.org/mod_mbox/dolphinscheduler-dev/202001.mbox/%3Ctencent_6DF620A15E4A57C43A349736CBCD15454907%40qq.com%3E&amp;amp;gt
&amp;gt; ;
&amp;gt;
&amp;gt;
&amp;gt; I did some effects with this, and there are some problems now:
&amp;gt; 1. I did't find checkstyle config file for Ali code style, which means 
we
&amp;gt; need to&amp;amp;nbsp;
&amp;gt; create these rules by ourselves.
&amp;gt; 2. Another possible way is to use PMD, but the PMD implementations has
&amp;gt; fewer&amp;amp;nbsp;
&amp;gt; rules than IDEA plugin or Eclipse plugin.&amp;amp;nbsp;
&amp;gt; For example indent check is not implemented by PMD, which I think
&amp;gt; is unacceptable.
&amp;gt; You can see The PDM implementations and rules&amp;amp;nbsp; at
&amp;gt; <https://github.com/alibaba/p3c/tree/master/p3c-pmd&amp;amp;gt;
&amp;gt;
&amp;gt;
&amp;gt; So, is there anyone can help solve these problems??
&amp;gt; Or we have to reconsider Google code style.

Reply via email to