careful-address-89803
05/18/2023, 2:55 AM./pants lint testprojects/src/helm:: -- --debug
? ArgsListOption
will handle the passthrough, I'm hoping to have some existing code for passing different args to subtasks for where it makes sense. For a concrete case, for deploying helm charts, we render the chart with helm template
and then deploy it with `helm upgrade`; some args are only valid for the upgrade
part, and we can filter them out of the ones we pass to template
.
Currently we only pass them through to upgrade
, which makes sense for, ex, --force
. But it's possible for the template
step to fail, and then --debug
would be helpful. See https://github.com/pantsbuild/pants/issues/18089careful-address-89803
05/18/2023, 3:03 AMwitty-crayon-22786
05/18/2023, 3:37 AMrun
, test
, rather than necessarily for lint
? it basically just needs to be unambiguous “who” will consume them.careful-address-89803
05/22/2023, 4:41 PMcareful-address-89803
05/22/2023, 7:20 PM--debug
so we add it to all steps. I think that's sufficient to address 99% of the usefulness without writing a fun little parser. Passing the other global Helm arguments requires more machinery and I don't think it would be that usefulwitty-crayon-22786
05/22/2023, 7:33 PM