Get rid of option_helpers.py.

Review Request #1439 — Created Dec. 6, 2014 and submitted

benjyw
pants
d420237...
pants-reviews
lahosken, zundel

It was redundant with global_options.py.

This allowed us to make doc gen for global options
more uniform with that for task-specific options.

CI passed.

Manually generated docs and eyeballed them.

LA
  1. Yay for cleaning this up.

  2. 
      
ZU
  1. 
      
  2. src/python/pants/option/options.py (Diff revision 1)
     
     

    I don't understand this line.

    1. GLOBAL_SCOPE is defined on ArgSplitter (and needs to be). But external code should only interaction with the Options class, so this denormalizes that constant onto Options.GLOBAL_SCOPE.

  3. 
      
ZU
  1. Ship It!

  2. 
      
BE
Review request changed

Status: Closed (submitted)

Change Summary:

Submitted as 07d517a88c330102d10753b1ad118c22c335737c.
Loading...