Properly lazily import shutil

[ofek]

I agree with the intention to lazily import but it wasn't actually happening. Locally I get the following on Python 3.12:

❯ python -m timeit -n 1 -r 1 "import shutil"
1 loop, best of 1: 8.09 msec per loop

[ofek]

Okay, that's everything!

[davidism]

Can you target this at stable instead?

[davidism]

There's a test that checks what modules are imported, you can probably remove shutil from that list as well.

[ofek]

I changed to stable and updated that test! If it's fine with you, I'm interested in the coming weeks/months making sure that everything (within reason) is lazily imported.

[davidism]

Definitely, it's a bit annoying to code, but I think the speed is worth it for the CLI. We want to distinguish between what's going to be imported no matter what (general Python startup / used everywhere anyway) versus what only comes up in certain use cases. That's what that import test is for, so we know if something gets imported unintentionally, and can review imports from time to time.