This repository has been archived by the owner on Mar 7, 2024. It is now read-only.
fix: rollup watch #148
Closed
fix: rollup watch #148
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
merge remaxjs master
修复app.config.js无法watch问题, 现在删除page文件夹后也会触发build的错误提示同时增加page路径的验证
|
是可以解决这个 #45 issue 下几个问题吗 |
|
上边两个问题是可以解决的，目前监听的是import文件以及appconfig文件，src下边全部文件的包括没有引入的文件吗，没有引入的文件删除新增这些操作，这种是不是打包会太频繁了？ |
|
@Crayonn 因为我们支持原生代码，原生代码是没有import的。所以需要监听所有文件。 新增 page，也会动态加入 entry 吗？ |
|
在app config中新增的page是可以的。可以做到监听原生模块的代码，将模块中的特殊文件集合做成配置吧，这样可以控制一点，因为这部分文件是需要重新编译整个项目的。 |
👍 晚上我跑起来试试看 |
|
不要 mege master，用 rebase 的方式 |
|
好的，我稍晚点把分支整理一下，正好我把特殊的那块做成配置吧，这样会更好一点 |
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 26, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 26, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 28, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 29, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 30, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 30, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 30, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 30, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 30, 2019
Crayonn
added a commit
to Crayonn/remax
that referenced
this pull request
Aug 30, 2019
Darmody
pushed a commit
that referenced
this pull request
Aug 30, 2019
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
修复app.config.js无法watch问题, 现在删除page文件夹后也会触发build的错误提示同时增加page路径的验证
不知道我这种修改方式合理不。