zfdx123 / build-k40-ksu

自动编译k40内核
19 stars 36 forks source link

支持参数从action输入 #2

Closed powerAn2020 closed 1 year ago

powerAn2020 commented 1 year ago

已实现参数输入

zfdx123 commented 1 year ago

workflow_dispatch的inputs可以直接定义默认值我记得,就不要env再次判断了,这样不合理我觉得

zfdx123 commented 1 year ago

直接使用 default: 'xxx' 来做这个事情

powerAn2020 commented 1 year ago

额,大佬,但是我的本意是需要保留从vars取值,如果设置default那么将始终无法从vars里取值,因为我个人打包的时候并不是每次都是填写action input 。

powerAn2020 commented 1 year ago

目前的处理流程是 有input就优先使用输入值,如果没有输入值就取vars变量值,如果都没有就取默认值

zfdx123 commented 1 year ago

设置def你也不需要每次添加数据啊,你不改就是你原来的值,和你判断一次再用vars有什么区别呢

powerAn2020 commented 1 year ago

我试过了🤣,设置默认值之后,哪怕全删不填写,input取值也是default的值

zfdx123 commented 1 year ago

你清空不改就是def,改了就是新值

zfdx123 commented 1 year ago

我已经测试了。用def没有问题

powerAn2020 commented 1 year ago

你清空不改就是def,改了就是新值

是的,我试的时候就是这样,所以后来才改成引入env来存储默认值,我举个例子,我fork了您的内核源码,加入了docker相关的打包配置,我现在只有两种选择,要么每次打包都输入input值,要么修改yml文件中定义的default值,个人感觉就好麻烦。我其实也不想用env,主要是表达式中无法嵌套表达式,没有找到替代方法,主要想实现优先input,其次vars,最后在默认

zfdx123 commented 1 year ago

23333,这似乎让我有点不能理解,因为你已经fork了库,您完全可以将def默认值修改为您想要的,如果有别的需求可以再input进行操作

powerAn2020 commented 1 year ago

额,主要原因是我同步代码都是在github网页上执行sync fork,一旦修改fork后的default,这将导致冲突而没法合并,只能先回滚到您的版本,然后再次改回我需要的default值。

zfdx123 commented 1 year ago

就目前来看,这个库除了更新boot别的应该是不会更新了

powerAn2020 commented 1 year ago

哈哈,那就先关闭了😁