Are there plans to create CLI shortcuts like

--no-process-execution-local-cleanup

->

-n

? 😗

you can add that to your own

pants.toml

I was going to say… but it seems that the alias feature does not work for engine flags.. (as it had no effect for me.. if that is what it was)

In specific cases that are universally useful we can create hardwired shortcuts in the code, but I imagine it would be hard to come to consensus on what those should be... Hopefully the alias functionality will serve?

I would also still like to rename the option to be --no-process-cleanup. We lose namespacing with other --process-execution args like --process-execution-local-parallelism. But because this cleanup option should be set via CLI arg rather than config file, I don't think that actually matters. You're not usually specifying those other options at the same time with CLI; namespacing doesn't buy us much

shouldn’t it be

--no-process-cache

? otherwise there’s no symmetry in what parts are dropped from the existing flags.. 😉

I don't think so. We have

--remote-cache-{read,write}

, so it's important that we distinguish between local vs remote here. Whereas with "cleanup", I don't think that's very relevant to disambiguate We could say

--no-process-local-cache

. But that's more verbose and we don't include

process

with the

--remote

options

Well, I think symmetry is still at play, only it has more inputs than what I was looking at here 🙂

https://github.com/pantsbuild/pants/pull/13769 @modern-wolf-36228 feel free to weigh in directly on that PR 🙂 Thanks for the feedback!