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

[询问] 请问是否能加入 @uni-ku/root 至 插件列表? #60

Closed
3 tasks done
Skiyee opened this issue Aug 23, 2024 · 8 comments · May be fixed by #72
Closed
3 tasks done

[询问] 请问是否能加入 @uni-ku/root 至 插件列表? #60

Skiyee opened this issue Aug 23, 2024 · 8 comments · May be fixed by #72

Comments

@Skiyee
Copy link
Member

Skiyee commented Aug 23, 2024

对问题的清晰和简明的描述

@uni-ku/root: https://github.com/uni-ku/root

目前能够支持 cli 和 hbx 创建的项目,基础功能已健全

该插件与 layouts 功能上可能会有点重叠,询问大佬是否能够加入

推荐的解决方案

@uni-ku/root 添加入到插件列表

替代方案

No response

额外上下文

No response

检查

@FliPPeDround
Copy link
Member

当然可以,非常欢迎
期待你的PR

只是有一个小小的疑问**.ku.vue, ku在这里的设计是出于什么考虑?
**.root.vue或者**.global.vue从语意上,个人觉得更清晰一些

@Skiyee
Copy link
Member Author

Skiyee commented Aug 23, 2024

当然可以,非常欢迎 期待你的PR

只是有一个小小的疑问**.ku.vue, ku在这里的设计是出于什么考虑? **.root.vue或者**.global.vue从语意上,个人觉得更清晰一些

感谢!❤

root.vue 是更好的选择,编写时没考虑那么多,变得一发不可收拾 😥

@FliPPeDround
Copy link
Member

FliPPeDround commented Aug 23, 2024

当然可以,非常欢迎 期待你的PR
只是有一个小小的疑问**.ku.vue, ku在这里的设计是出于什么考虑? **.root.vue或者**.global.vue从语意上,个人觉得更清晰一些

感谢!❤

root.vue 是更好的选择,编写时没考虑那么多,变得一发不可收拾 😥

要是你认为有必要,在还没有推广,并没有多少用户的时候,是最适合修改的时候
随便一提,uni-ku/root这个名字并不符合vite插件的社区命名规范虽然不是很重要,但一个能让用户一眼就看出功能的命名,听起来会更合适。
当然保持原样,同样也欢迎你的pr ;)

@Skiyee
Copy link
Member Author

Skiyee commented Aug 29, 2024

要是你认为有必要,在还没有推广,并没有多少用户的时候,是最适合修改的时候 随便一提,uni-ku/root这个名字并不符合vite插件的社区命名规范虽然不是很重要,但一个能让用户一眼就看出功能的命名,听起来会更合适。 要是你认为有必要修改的话,可以趁还没有推广前改改。当然保持原样,同样也欢迎你的pr ;)

在下一版本进行关于 App.ku.vue 命名更改的破坏性更新后,再提交相关的PR

包的命名会继续保持,我理解是这样的,uni 作为前缀就指定这是一个uniapp相关的包,鉴于部分使用者并不清楚vite插件命名规范,直接明了的命名,可能会使其更加容易理解

@FliPPeDround
Copy link
Member

好呢,;)
对了,大佬 有没有测试,App.ku.vue这样的文件命名方式会不会和vite-plugin-uni-platform冲突

@Skiyee
Copy link
Member Author

Skiyee commented Aug 29, 2024

好呢,;) 对了,大佬 有没有测试,App.ku.vue这样的文件命名方式会不会和vite-plugin-uni-platform冲突

应该不会,uni-platform库 是取 uni-env库 当前所在的 platform 作为识别扫描的

提交PR前我测试一下确保不产生冲突

@Skiyee
Copy link
Member Author

Skiyee commented Sep 30, 2024

由于时间关系,暂时先把该issue关闭推迟 ❤

@Skiyee Skiyee closed this as completed Sep 30, 2024
@FliPPeDround
Copy link
Member

由于时间关系,暂时先把该issue关闭推迟 ❤

没事,不急,挂着吧

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 a pull request may close this issue.

2 participants