Shouldn't the SelfCommand instead be made not an EnvCommand?

Or perhaps the whole notion of EnvCommand wants abandoning and commands should instead get a requires_venv(self) -> bool or similar.

Shouldn't the SelfCommand instead be made not an EnvCommand?

SelfCommand and its derived classes for the several self commands inherit EnvCommand to mirror the functionality from the command without self.

Or perhaps the whole notion of EnvCommand wants abandoning and commands should instead get a requires_venv(self) -> bool or similar.

Yes, I had something similar in mind some time ago. But this would require a bigger refactoring. Let's do it at a later point.

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.