Skip to content

Contributing guide

赵鹏程(珵之) edited this page Jan 26, 2024 · 10 revisions

贡献代码

这篇指南会指导你如何为 @alifd/next 贡献代码,请你在提 issue 或者 pull request 前花几分钟来阅读下。

透明的开发

所有的开发工作都发生在 Github 上,不论是核心团队成员还是外部的贡献者,pull request 后都会走相同的 review 流程。

分支管理

  • 日常 bug 修复、优化、小功能,向 master 提交 pull request

v1 版本停止于 1.27.x,并停止新增组件,为 2.x 做准备

创建 Issue

我们使用 GitHub Issues 来做 bug 和 feature 的追踪。在提交 Issue 时请选择 Bug report 或 Feature request 对应的模版,填入必要信息,以帮助我们快速定位以及解决问题。

另外,在你报告一个 bug 或提交一个 feature 之前,请先确保已经搜索过已有的 issue。

Pull Request

我们会关注所有的 pull request,review 以及合并你的代码,也有可能要求你做一些修改或者告诉你我们为什么不能接受这样的修改。

在你发送 Pull Request 之前,请确认你是按照下面的步骤来做的:

  1. 基于上文所述正确的分支做修改

  2. 在项目根目录执行 npm install,安装所有开发依赖

  3. 如果你想修改 Button 组件的代码,在项目根目录执行 npm run dev button,会自动为你启动浏览器并打开 demo 页面

  4. 如果你修复了一个 bug 或者新增了一个功能,请确保写了相应的测试,可以通过在根目录执行 npm run test 来启动所有组件的测试,在开发过程中可以用 npm run test button 来运行指定组件的测试,如果你想在浏览器中调试测试用例可以使用 npm run test:head

  5. 请确保你修改的代码通过了 eslint 和 stylelint 检查,我们会在 precommit 阶段中对你加入到 git 缓存区的代码文件自动执行 lint。

  6. 请确保你的 git 提交信息格式符合我们的以下要求:格式为 <type>(<scope>): <subject>type必填,可选值包括:build|chore|ci|docs|feat|fix|perf|refactor|revert|style|test|temp;scope 必填,填写具体修改的组件名,如 feat(Menu),如进行批量修改或其他改动,可写为 *,如 chore(*)subject 必填,使用英文,小写开头,如若关闭 issue,可写为,如 fix(Menu): resolve xxx issue, close #12,我们会在 commit message 阶段对提交信息进行检查

  7. 如果一次 pull request 涉及到多次提交,请对提交记录进行 rebase 操作,缩减为一条提交记录

开发流程

环境准备 node 16npm 8

  1. 将代码 fork 到自己的 github 仓库 (在 https://github.com/alibaba-fusion/next 上,点击右上角 fork 按钮)
  2. 使用 Gitpod, 一个源自 GitHub 的免费在线代码编辑工具

Open in Gitpod

或者下载代码到本地: 你可以在 https://github.com/[your account]/next 找到刚 fork 的代码

git clone git@github.com:[your account]/next.git
  1. 进入刚克隆的目录,为了后续开发、同步主仓库,先设置 upstream
cd next
git remote add upstream git@github.com:alibaba-fusion/next.git
  1. 开发时不要直接在 fork 库的 master 上进行开发,fork 库的 master 应当仅用来同步主库的 master。将主库主干代码同步至 fork 库:
git pull upstream master
git push
  1. 基于更新后的 fork 库 master,创建分支进行开发
git checkout -b fix-issue-100

其中 4、5 是每提交一个 pr 就需要执行的操作。

在完成上述操作并且使用 npm install 安装完依赖后,你还可以运行下面几个常用的命令:

  • npm run start date-picker 启动指定组件的调试页面

  • npm run test:head 在浏览器内调试测试用例

  • npm run test date-picker 启动指定组件的测试

  • npm run test 启动所有组件的测试

  • npm run api date-picker 根据代码和注释,自动更新指定组件的 API 文档

  • npm run check 检查所有组件的 TS 类型、eslint、stylelint

  • npm run check date-picker 检查指定组件的 TS 类型、eslint、stylelint

  • npm run build 构建组件库,生成 es、lib、types、dist、demos、compiled_docs 目录

  1. 开发完成后按照要求补充单元测试、运行单元测试、编写语义化的 commit 信息后,进入 https://github.com/[your account]/next 找到 New pull request按钮提交 PR

  2. 维护者可能会有一些修改建议,开发者可能需要根据修改建议反复修改代码。最终由组件库维护者合并 PR,在下次发布时本次修改的功能生效。

开发注意事项

  1. Fusion Next 作为前端组件库支持 SSR 因此需要做到:
    • 尽量避免在 constructor getDerivedStateFromProps componentWillMount (deprecated) 这些生命周期中,使用 window localStorage sessionStorage document navigator 等客户端变量;
    • 必须使用的话,客户端端对象的判断用 typeof
      if(window && window.autoScroll)
      =>
      if(typeof window != undefined && window.autoScroll) )
      
    • 避免往 window 等全局对象挂载定时器
    • 避免 random() 等不确定性输出 (输出结果可预期,不依赖于环境等)
  2. sass 颜色变量计算的结果,需要以 $color-calcualte-开头,写到组件的 variable.scss 中 (不能写到 main.scss 中),参考考Search组件,#1029
  3. 所有 sass 计算需要被 calc 包裹

《Tips for server-side rendering with React》 https://itnext.io/tips-for-server-side-rendering-with-react-e42b1b7acd57

发布周期

遵循 Semantic Versioning 2.0.0 语义化版本管理策略:

z 位:每双周四会进行日常 bug 修复版本的更新,紧急问题不受此限制,可以随时发布

y 位:不定期发布一个带有新特性的向下兼容的版本,一般周期一到两个月

x 位:包含有 break change 变更的大版本,一般周期一到两年

我们正在做并欢迎 PR 的清单

  1. Easy to solve issues
  2. 组件技术升级