Skip to content
This repository has been archived by the owner on Apr 17, 2023. It is now read-only.

Fully Genericize the Explain Command #351

Open
6 tasks
michaeltlombardi opened this issue Feb 15, 2022 · 0 comments
Open
6 tasks

Fully Genericize the Explain Command #351

michaeltlombardi opened this issue Feb 15, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@michaeltlombardi
Copy link
Contributor

As with build, the explain command can be fully genericized for reuse without re-implementation in other projects such as PRM.

Acceptance Criteria

  • Convert the var declaration in the explain command to a ExplainCommand struct
  • Add the ExplainCommandI interface which must implement CreateCommand()
  • Update CreateCommand(), preExecute(), validateArgCount(), flagCompletion(), and execute() to operate on the ExplainCommand struct, updating references from the variables to the struct's properties
  • Remove the logic from CreateCommand which instantiates the docs file system; this should be passed when the struct is instantiated in main
  • Remove any references to pct, genericizing where needed
  • Ensure tests continue to pass
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant