I was confused by timezones, I thought tomorrow meant today (Tuesday 15th), so I seems I missed the develop meeting. I would have liked to talk about my two open PR: • github.com/pantsbuild/pants/pull/23668 • github.com/pantsbuild/pants/pull/23620

Ah yes, sorry those took a while to get to. My personal life is a bit chaotic right now (in a good way!) I just approved and merged https://github.com/pantsbuild/pants/pull/23668 after going over the design doc and commenting there.

hey, sorry to hear life is chaotic in a good way. I do appreciate working this kind of open source project is a lot of effort for not much reward.

Yeah, I think it looks fair, I'm wondering if you explored the possibility of having a threshold for when this kicks in, to avoid the extra Digest write on everything that don't have a very long arg list, or if we should make it configurable if there would be any compatibility issue for someone with the arglist via file? I'm just wondering because there so much going into 2.34 now that it is just a slight concern if it could risk blocking that upgrade for someone, especially if they're not affected by the arglist issue

the change to only use the file over a certain threshold seems quite simple so I added it to the PR: github.com/pantsbuild/pants/pull/23620/…/123ae0896e5c…

cc @fierce-truck-19259