Tweak to option default extraction for help display.

Review Request #2640 — Created Aug. 16, 2015 and submitted

benjyw
pants
69eaff2...
pants-reviews
jsirois
Previously a boolean option with no explicit default displayed
as 'None' instead of 'False' or 'True' as appropriate.

Added a bunch of tests for this, and also for flag scoping.

CI in flight: https://travis-ci.org/pantsbuild/pants/builds/75834137.

JS
  1. 
      
  2. You might just:

    r = lambda value: mock.Mock(value=value)
    
    1. That is nicer. Changed.

  3. 
      
BE
JS
  1. Ship It!
  2. 
      
BE
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as ba6dc819ff45e7c616a8986fe3204fc135866450.

Loading...