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

feat/add watch build mode #396

Merged
merged 6 commits into from
Nov 14, 2022
Merged

feat/add watch build mode #396

merged 6 commits into from
Nov 14, 2022

Conversation

yangxiaolang
Copy link
Collaborator

@yangxiaolang yangxiaolang commented Nov 7, 2022

watch build 用于 AUI 的类库调试,方便直接在引入 AUI 的项目中对 AUI 直接进行开发调试,AUI 的改动能实时更新到引入项目。

除了通过yarn link软链接的方式,也支持构建产物直接拷贝到目标项目

直接在 build:watch 命令后加上调试项目的路径即可,会自动检测 node_modules 目录,copy release 到目标项目node_modules@alauda/ui( aui的package-name决定 ) , 目标路径下没有node_modules会报错

这种做法相比yarn link软链接的好处是,相当于直接在目标项目处建了备份,目标项目的angular.jsonNODE_OPTIONS不需要设置preserveSymlinks
坏处就是每次watch build完都会把release完整拷贝到目标项目处一份

@changeset-bot
Copy link

changeset-bot bot commented Nov 7, 2022

⚠️ No Changeset found

Latest commit: 859b718

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

scripts/build.js Outdated Show resolved Hide resolved
@fengtianze
Copy link
Collaborator

readme加个文档

@fengtianze fengtianze merged commit cf95fbf into master Nov 14, 2022
@fengtianze fengtianze deleted the feat/add-watch-build-mode branch November 14, 2022 02:55
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