You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Fix dependency issue with cmd2-ext-test plugin #1517
Now that we put out a 3.0.0b1 beta release for cmd2, much of our automatic CI/CD is failing due to unsatisfiable requirements related to the cmd2-ext-test plugin.
Regardless of the path forward, the current situation we have where testing cmd2 requiring a plugin which itself depends on cmd2 is an abomination of circular dependency that needs to end. A plugin which depends on cmd2 should not IMO be part of the main repository.
I see three general possible paths forward:
Deprecate the cmd2-ext-test plugin, delete the code for it, and include the mixin class in our isolated tests
Move the cmd2-ext-test plugin to a separate repository and modernize it so it works with cmd2 3.x
Bake the mixin class into cmd2 itself so it is available to all users and deprecate the plugin
One option is to document for users how they can create this mixin tests for their own testing ease.
PR #1518 effectively takes the first approach outline and deletes the cmd2-ext-test plugin and copies the ExternalTestMixin class to the one set of tests which use it.
I'd be happy to explore the 2nd or 3rd option, but the first option is effectively a prerequistie.
Activity
github-actions commentedon Sep 16, 2025
🤖 Hi @tleonhardt, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.
tleonhardt commentedon Sep 16, 2025
@kmvanbrunt @anselor Do either of you have any thoughts or opinions here?
tleonhardt commentedon Sep 16, 2025
PR #1518 effectively takes the first approach outline and deletes the
cmd2-ext-testplugin and copies theExternalTestMixinclass to the one set of tests which use it.I'd be happy to explore the 2nd or 3rd option, but the first option is effectively a prerequistie.