pants
pants copied to clipboard
Mark all Python tools as exportable
Following to #20730 , this MR flags all Python tools as exportable. It omits some internal tools (for example, parsers for dependency inference).
I think there's a risk that someone tries to export a Python-based tool without activating the Python backend. I also want to look into a few instances where some lockfile rules are registered and see if those registrations are still necessary.
When you have a chance, please merge main
(or rebase onto it) and add some release notes to docs/notes/2.22.x.md
. See https://github.com/pantsbuild/pants/discussions/20888 for more info.
Also, other than that, is this waiting on anything other than hitting the merge button?
I think we're good to go. I thought that it might be nice to have a toggle to turn this off, in case it causes problems during migration or people don't like the UX of having all resolves suggested together. I'm not sure that's worth the effort, though
Is that on your radar?
It probably should have been, the purpose of this MR was to remove those 😅. Thanks for pointing them out, I've updated them
I thought that it might be nice to have a toggle to turn this off, in case it causes problems during migration or people don't like the UX of having all resolves suggested together. I'm not sure that's worth the effort, though
We can see how it goes in dev release testing and retro-fit if required, I imagine?
Makes sense