Create a mixin for "quiet tasks".

Review Request #760 — Created July 26, 2014 and submitted

benjyw
pants
pants-reviews
patricklaw
This separates the two concerns formerly handled by ConsoleTask:
being quiet, and being a task whose primary job is to print to the console.

Now you mix in QuietTaskMixin for the first, and extend ConsoleTask
(which itself mixes in QuietTaskMixin) for the second.

This rids us of silliness such as the clean-all task having a flag
to specify its output separator.
CI passed: https://travis-ci.org/pantsbuild/pants/builds/30891653

Also local runs of all unit and integration tests.
LA
  1. 
      
  2. While you're in there, please add a docstring. Maybe something like what I heard in a lovely change description lately:
    
    """A task whose primary job is to print to the console."""
  3. 
      
BE
Review request changed

Status: Closed (submitted)

Loading...