dojo / cli

:rocket: Dojo - command line tooling.
http://dojo.io
Other
26 stars 34 forks source link

Unify execution #303

Closed devpaul closed 4 years ago

devpaul commented 4 years ago

Type: cleanup

The following has been addressed in the PR:

Description:

The code to execute a command's validation and run phases exists in two places: once for the default command and once for other commands. It's too easy to make a change to how a command is executed in one spot and not the other.

This PR unifies command validation and execution for both default and other commands.

agubler commented 4 years ago

@devpaul I think this looks reasonable, just got a conflict that needs resolving.