Remove all direct config uses from jar_publish.py.

Review Request #1844 — Created Feb. 28, 2015 and submitted

benjyw
pants
9d44bb6...
pants-reviews
dturner-tw, ity, jsirois, zundel
This also required switching the JarPublish unit tests to use
the new-style TaskTestBase base class.

CI passes: https://travis-ci.org/pantsbuild/pants/builds/52487799

ZU
  1. 
      
  2. I feel like the docs for this and --publish-extras are weak here in the help. Can we describe the format of the dict or provide a URL for more info?

    1. Here's the link with the run-down on repos: https://pantsbuild.github.io/publish.html#authenticating-to-the-artifact-repository
      And customizing publish with extras: https://pantsbuild.github.io/dev_tasks_publish_extras.html
      
      The "we"s are killing me guys.  This is most definitely a request for Benjy to fix these things up.
    2. Added URLs.

  3. 
      
DT
  1. 
      
  2. Can we be consistent here between dashes and underscores?

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

Status: Closed (submitted)

Change Summary:

Submitted as d71c53e4ef8c779d08afe546adb820618335f5f1.

ZU
  1. Ship It!
  2. 
      
Loading...