gulp-alias-combo icon indicating copy to clipboard operation
gulp-alias-combo copied to clipboard

代码中含有define的方法,也会被替换为 gmu.define( '别名','Dialog', {})

Open sunlanlisa opened this issue 9 years ago • 10 comments

define(function(require){ gmu.define( 'Dialog', { options: {} }) })

会变成这样的 导致不可用,最后会被替换为(gmu-dialog是取得路径别名) define('gmu-dialog', function(require){ gmu.define('gmu-dialog', 'Dialog', { options: {} }) })

有什么解决方案吗,大神求助。。。

sunlanlisa avatar Oct 11 '16 09:10 sunlanlisa

这个情况目前确实有这个问题,我晚上改下代码,兼容下 xxx.define 这种情况

[email protected]

发件人: sunlanlisa 发送时间: 2016-10-11 17:00 收件人: PeterMu/gulp-alias-combo 主题: [PeterMu/gulp-alias-combo] 代码中含有define的方法,也会被替换为 gmu.define( '别名','Dialog', {}) (#4) define(function(require){ gmu.define( 'Dialog', { options: {} }) }) 会变成这样的 导致不可用,最后会被替换为(gmu-dialog是取得路径别名) define('gmu-dialog', function(require){ gmu.define('gmu-dialog', 'Dialog', { options: {} }) }) 有什么解决方案吗,大神求助。。。 — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

PeterMu avatar Oct 11 '16 09:10 PeterMu

😉,谢谢大神~🐂

sunlanlisa avatar Oct 11 '16 09:10 sunlanlisa

升级下 gulp-alias-combo 到最新版本(v0.3.1),问题就解决了。

PeterMu avatar Oct 11 '16 15:10 PeterMu

我安装了,但是出现了问题,所有的define都没有添加别名呢。。这种也没添加,define(function(require){}} 导致虽然打包放到了一个文件里,但是找不到

sunlanlisa avatar Oct 12 '16 02:10 sunlanlisa

😭

sunlanlisa avatar Oct 12 '16 02:10 sunlanlisa

额?你看下 test 目录,test 目录里有测试用例,在 test 目录里,运行 gulp 你看下结果是否正常

PeterMu avatar Oct 12 '16 02:10 PeterMu

@sunlanlisa 我又测试了一遍,测试用例我跑着没问题,如果你方便的话,可以把构建有问题的部分代码和 gulpfile.js 发送到我的邮箱([email protected])。

PeterMu avatar Oct 12 '16 03:10 PeterMu

找到原因了,如果在s前面有注释,就不行。你的test用例也一样哦

/*** Created by xxx ***/

sunlanlisa avatar Oct 12 '16 04:10 sunlanlisa

sorry, 是匹配 define 的正则有问题,我还没想到更好的正则,但是开头有注释的这个情况已经修复了,更新到最新版本(v0.3.2),非常感谢耐心反馈!

PeterMu avatar Oct 12 '16 06:10 PeterMu

恩恩,好的,谢谢~~O(∩_∩)O~

sunlanlisa avatar Oct 12 '16 06:10 sunlanlisa