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

缺失 https://tw-cpl.netlify.app/repo/TWaddle_ListTree/__meta__.json #152

Open
linonetwo opened this issue Oct 13, 2023 · 13 comments
Open

Comments

@linonetwo
Copy link
Collaborator

在安装 Aggregation-people 的时候

这个 TWaddle_ListTree 是啥,从没听过

@linonetwo
Copy link
Collaborator Author

xp 里有,不过还是找到了原始出处装了 f112df3

@Gk0Wk
Copy link
Contributor

Gk0Wk commented Oct 13, 2023

为什么能搜到但是meta没有啊,按理来说要没有都没有

@Gk0Wk
Copy link
Contributor

Gk0Wk commented Oct 13, 2023

是不是因为原插件有但是 依赖的插件 没有

@linonetwo
Copy link
Collaborator Author

是的,这是 xp 的可选的插件,之前没装过就没加。

但是现在列在 dependent 里的都必装了,所以就发现缺失了。

@oeyoews oeyoews reopened this Oct 14, 2023
@oeyoews
Copy link
Contributor

oeyoews commented Oct 14, 2023

img

  • 如果依赖缺失, 可以跳过依赖, 提示用户(cpl现在还没有上传cdr/markdown-more插件, 所以造成安装失败)

@linonetwo
Copy link
Collaborator Author

你的插件依赖了,就顺便添加一下嘛

@Gk0Wk
Copy link
Contributor

Gk0Wk commented Oct 14, 2023

img

  • 如果依赖缺失, 可以跳过依赖, 提示用户(cpl现在还没有上传cdr/markdown-more插件, 所以造成安装失败)

今天改一下

同时:如果cpl 本地已经安装了依赖,就不去递归检测那个依赖

@oeyoews
Copy link
Contributor

oeyoews commented Oct 14, 2023

其实写成可选依赖更合适

@Gk0Wk
Copy link
Contributor

Gk0Wk commented Oct 14, 2023

原本的tw没有soft-dependents,我们可以加一下

@linonetwo
Copy link
Collaborator Author

应该是 optionalDependency ,参考 npm 的

@oeyoews
Copy link
Contributor

oeyoews commented Oct 15, 2023

父插件可以指定某个固定的依赖插件版本吗

image

@Gk0Wk
Copy link
Contributor

Gk0Wk commented Oct 15, 2023

可以,但是这就超出原来的tw太多了

@linonetwo
Copy link
Collaborator Author

这就不太方便了,难道要用户的笔记网页上放两个同样的插件文件吗?这个空间占用成两倍了。

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

No branches or pull requests

3 participants