CP-822 CLI improvements & cleanup.

[evanweible-wf]

Issue

• Initial implementation could use some minor cleanup and improvements to the CLI.

Changes

• Consolidated utils, reporter, and TaskProcess class into single util.dart entry point
• init task now includes a link to the dart_dev repo as a comment in the generated file
• analyze and format tasks updated to include bin/ directory
• CLI Improvements:
  • Catch the FormatException when an unsupported arg is used and display a helpful message along with the usage text instead of the stack trace
  • examples task now checks for the example/ directory and displays an error message if it doesn't exist
  • format task now verifies that dart_style is an immediate dependency and will display a helpful error message if that's not the case
  • test task now verifies that test is an immediate dependency and will display a helpful error message if that's not the case

Areas of Regression

• n/a

Testing

• ddev --unsupported should display an error message stating that the --unsupported option is not supported in addition to the standard usage text instead of displaying the FormatException stack trace
• examples
  • warns if example/ directory doesn't exist
• format
  • warns if dart_style isn't an immediate dependency (remove dart_style line from pubspec.yaml to test)
• init
  • includes a link to dart_dev repo in generated tool/dev.dart file
• test
  • warns if test isn't an immediate dependency (remove test line from pubspec.yaml to test)

Code Review

@trentgrover-wf
@maxwellpeterson-wf
@dustinlessard-wf
fyi: @jayudey-wf

[trentgrover-wf]

+1

[maxwellpeterson-wf]

+1

[maxwellpeterson-wf]

@jayudey-wf ready for merge

[jayudey-wf]

QA Resource Approval: +10

• Testing instruction
• Dev +1's
• Dev/QA +10 with detail of what was tested
  • all testing suggestions work as expected
• Unit test created/updated
• All unit tests pass

Merging into master.