Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fairydust 的来由与定位 #1

Open
lifesinger opened this issue Aug 16, 2013 · 8 comments
Open

fairydust 的来由与定位 #1

lifesinger opened this issue Aug 16, 2013 · 8 comments

Comments

@lifesinger
Copy link
Contributor

很同意 @hotoo 的建议:

这个应该跟质量工具结合起来,质量工具是针对这些规范的实现。
一般来说,不符合编码规范中定义的属于错误,不符合风格指南属于警告。

所以不要是干干做一个编码规范,远尘搞一套质量标准。。。
建议在 Alipay 下建一个仓库,大家 fork & pull。大家觉得可以通过的就 merge。

总体上应该可以包含以下几点:

编码规范:一定要遵守的行为规范。比如单个文件的缩进风格必须一致,可以是空格或 tab 缩进。
风格指南:推荐的代码风格,仅作为指导方针。比如空格缩进宽度建议是 2 个或 4个。
最佳实践:从实践中积累的经验,用于指导编程实践。比如设计模式什么的。

@lifesinger
Copy link
Contributor Author

煤灰精灵(Fairydust),有沉淀的意味,只有新人小孩子才能看见,而且一般是在没人的角落。

@fool2fish
Copy link

冒顿取的这名字很有爱啊

ps:强烈建议把入乡随俗作为第一原则

@lizzie
Copy link

lizzie commented Aug 16, 2013

你们都好有文化啊.....

@popomore
Copy link

这个定位和周会说的又不一样了

@afc163
Copy link

afc163 commented Aug 16, 2013

这个库的名字为啥很土鳖的拆成了 fairy-dust?就好像 great-wall,git-hub, life-singer

@lifesinger
Copy link
Contributor Author

经小偏偏一点拨,觉得确实很土鳖,我合起来?有反对票没?

On Fri, Aug 16, 2013 at 4:28 PM, 偏右 [email protected] wrote:

这个库的名字为啥很土鳖的拆成了 fairy-dust?就好像 great-wall,git-hub, life-singer


Reply to this email directly or view it on GitHubhttps://github.com//issues/1#issuecomment-22753942
.

王保平 / 玉伯(射雕)
送人玫瑰手有余香

@hotoo
Copy link
Member

hotoo commented Aug 16, 2013

💯

@lifesinger
Copy link
Contributor Author

看看尹曰有没有兴趣,一点讨论:

from 玉伯:

我们目前总的宗旨是 「入乡随俗」。但什么是 俗 呢?俗是习俗的意思,包括:

  1. 各种社区已经沉淀下来的规范(不同的「俗」,我们是否可以做些总结归纳,不需要 copy 过来,给出链接梳理就行)
  2. 我们自己推荐的「俗」
  3. 支付宝特定的「习俗」,比如 钩子规范、和业务相关的埋点规范等等

把这些「俗」整理出来后,我们再确定自己再哪个「乡」,这样才能真正把 入乡随俗 落地

以上是 编码风格 、规范 层面

有了以上这些规则,更进一步的就是做工具,当初我们做过一个工具(半成品):
https://github.com/totorojs/catbus

我更希望我们是否可以做成这种:

https://codeclimate.com/

尹曰可以先看下,这一块深入下来,还涉及究竟怎么做 code review(简称 CR)
CR 的工作内容部分,究竟哪些可以用工具自动化掉

支付宝目前的 俗 是 作者为大,没什么约定,长期还是不太好,至少我们还是得有自己的推荐的 俗(真正让大部分人认可的)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants