-
Notifications
You must be signed in to change notification settings - Fork 3k
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
2.0 无法打包通过 npm 引入的 wpy 文件 #2554
Comments
@Gcaufy @deepfunc 我尝试定位了一下错误原因。 对于 wepy/packages/cli/core/plugins/parser/wpy.js Lines 16 to 27 in 95911eb
发现其判断是否是 npm 引入的方式是判断其 而编译组件时触发上面这个钩子的相关代码: wepy/packages/cli/core/compile.js Lines 271 to 281 in 95911eb
传入的 正确的做法应该是直接读取 let context = {
file,
type,
npm: !!comp.npm // 此处
component: true
}; |
此处的属性设计在一开始没有做好,导致会出现修复了这种场景却导致另外一种场景出现问题。 看看这里是否可以用 test cases 来保证这个场景的正确性。 |
@Gcaufy 这些属性都分别代表什么意思?我现在看起来其实是比较乱的,也没有注释,只能根据上下文和变量名尽量去猜。 另外目前都有哪些场景?我应该看源码的哪个地方可以确定他们应该有的值? 测试用例我可以来写。 |
顶一个,遇到同样的问题,希望尽快解决 |
请问现在wepy2开发插件可以成功吗 |
请问你的问题解决了码 |
@lxf-soul 我弃坑了,改用 uni-app 了,现在还活着的跨端小程序框架也只剩这一个了 |
我用的原生写的包,然后再wpy里引入 |
看我也要跑了,这个框架坑太大了 |
1.7.X 版本是可以引用并正常打包通过 npm 引入的 wpy 组件的。
2.0 可以通过 npm 引入原生组件、但引入 wpy 组件却不能正常打包(具体现象是不能在输出目录中创建
$vendor
目录)。示例代码如下:
目前发现只需要改动
@wepy/cli/core/plugins/parser/wpy.js
中的第 25 行:修改为:
即可正常打包 wpy 组件,还未发现有其他副作用。
麻烦确认是否是 Bug。
The text was updated successfully, but these errors were encountered: