-
Notifications
You must be signed in to change notification settings - Fork 91
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
在测试中加入检查源码是否可以在 MCDR 中正常加载 #395
Comments
MCDR 本身没有这方面设计吧?好像有点麻烦 |
checkout,下载插件,然后启动 MCDR 就行了。可以检查 log 或者用另一个插件输出标准的已加载插件列表 |
这样的话出错了没法给出 exc 输出吧……? |
直接提示无法加载就行了,用插件设置 GitHub Output
|
那比如我在 on_load 检测到配置没更改就给你 raise 了,就只能去看 workflow 日志才知道了( |
倒是可以用一些怪方法手动检查 |
另外一个问题:如果有尚未入库的前置插件,也会导致检测失效 |
不影响,少量这类问题可以手动处理,这应该是极少的情况。对于大部分正常提交的插件,可以作为一道保险 |
我认为不太可行,原因如下
|
我的观点是,报告中加入插件是否已经提供有效 Release 是可以作为参考的,检查能否加载参考价值不高,而且处理不好的话挺危险的 |
假定大部分插件在正确安装依赖后是开箱即用类型的插件,这个检查的主要价值是:
基于上文的前提,目前应该有几个插件需要特定的配置,例如需要修改配置文件或第三方服务的插件,这其中的部分插件会选择拒绝加载,也有部分会关闭功能等待配置。 如果自动检查不通过,消息提示应当作为警告来考虑,可以提示作者,并且由维护者选择是否需要手动检查,以及防止作者仓促提交(遗漏信息或版本错误,用户下载后并不能开箱即用,例如有部分插件是不能自动生成配置文件的,需要用户手动下载,否则加载时会报错)。 提出这个建议是和贡献指南中的建议提供一个可用发行版配合的,可能收益并不会很大,作为一道额外的保险,是否加入请各位讨论斟酌。 |
这个没多大问题
要检查这个得多版本 MCDR 多次测试? |
有标记按标记测试,没有按一个较低的版本即可,比如多文件插件使用 2.0,单文件用 1.0。理论上现在所有多文件插件都应该标记 2.0 |
Python 的 dependency 是按照 PEP 508 来的,似乎很难做到“安装符合条件的最低版本” |
不需要找到,尽量低即可,因为没有标记,即为兼容所有版本,MCDR 目前没有标注 LTS 的版本,用旧一些的 LTS 版本也是可以的。
|
我的意思是,类似声明 >=2.1.0 的插件理论上应该在 2.1.0 测试,但 pip 会安装最新版本 |
这个是技术细节问题了,不在 issue 讨论范围内 |
在 PR check 测试中加入检查源码是否可以在 MCDR 中正常加载,顺便检查 requirements 和依赖。
Originally posted by @AnzhiZhang in #394 (comment)
The text was updated successfully, but these errors were encountered: