Migrate checkstyle's use of self.config to self.get_options() for fetching from pants.ini.
Review Request #1399 - Created Nov. 25, 2014 and submitted
The options names should be backwards compatible. However, the task doesn't seem to be wired in to any
goal at the moment, so I can't add a section name to the migration script.
|now i can actually test your changes, i see some problems: the logic should be if bootstrap-tools/suppression_files/configuration not specified in ...||Jin Feng|
now i can actually test your changes, i see some problems:
the logic should be if bootstrap-tools/suppression_files/configuration not specified in cmd line args, then the ones defined in pants.ini [checkstyle] section should be used.
However, your change seems to only use from cmd options values (or unless I don't understand the new options system, which actually would look at pants.ini as well?)
I wired checkstyle up in 'optional' backend, and confirmed the problems I outlined above. If I don't specify those args in cmd line, checkstyle is broken.