Skip to content

build: migrate to dumi & father-build #488

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

Closed

Conversation

PeachScript
Copy link

@PeachScript PeachScript commented May 12, 2020

Changes

  • Use dumi instead of storybook as the doc tool
  • Use father-build instead of father as the build tool to reduce install size

@PeachScript
Copy link
Author

test failed,有些奇怪,本地无法复现

@PeachScript
Copy link
Author

PeachScript commented May 12, 2020

test failed,有些奇怪,本地无法复现

走 npm 安装依赖复现了,估计之前能 pass 是 tnpm 拉平依赖导致的。
ci 挂掉的实际原因是 @umijs/test 和 umi-test 都装上了,但如果只安装 @umijs/test 的话,用例会挂一大片,test 估计有些迁移成本,还是复原为 umi-test 吧

@PeachScript PeachScript changed the title build: migrate to dumi & father-build & @umijs/test build: migrate to dumi & father-build May 12, 2020
"prepublishOnly": "npm run compile && np --no-cleanup --yolo --no-publish",
"lint": "eslint src/ examples/ --ext .tsx,.ts,.jsx,.js",
"test": "father test",
"test": "umi-test",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

father umi dumi 这里面三套东西。

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

目前是这样的,后面 dumi、@umijs/test 都可以作为 umi preset 挂载到 father 上,对用户来说会 all in father 了,比如 father test -- 等同于 --> @umijs/test,father dev -- 等同于 --> dumi

@jljsj33
Copy link
Member

jljsj33 commented Nov 16, 2020

冒个泡,赶紧

@PeachScript
Copy link
Author

已切 dumi 2 + father 4,PR 关闭

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

Successfully merging this pull request may close these issues.

3 participants