Avoid using $_ when loading the commands from namespaces

Open bbrtj opened this issue 1 year ago • 6 comments

Solution proposed in #2175

Polluting $_ can happen deep down in dependencies tree of the loaded command.

resolves #2175

May 17 '24 18:05 bbrtj

Bump, why isn't this merged? Seems like a low-hanging fruit.

Dec 09 '25 07:12 bbrtj

Nobody will even look at it before tests passed,

Dec 09 '25 10:12 kraih

I'm confused. Am I required to take any action here? Did the tests fail? They look like they never ran at all.

Dec 09 '25 11:12 bbrtj

No idea what went wrong i'm afraid, and nothing we can do about it.

Dec 09 '25 16:12 kraih

I rebased to latest main. Seems like workflows await approval now. Maybe that'll fix them.

Dec 09 '25 17:12 bbrtj

Most tests passed now. Windows test seems to be failing in other PRs too, regardless of their content.

Dec 10 '25 03:12 bbrtj