verify config by default

Review Request #3636 — Created April 1, 2016 and submitted

wisechengyi
pants
pants-reviews
benjyw, mateor, patricklaw, stuhood, zundel

Make --verify-config default True to better help folks catch errors.

Performance impact is minimal.

[tw-mbp-yic pants (master)]$ time for i in `seq 50`;do ./pants goals; done &> /dev/null

real    0m38.839s
user    0m22.951s
sys 0m12.697s


[tw-mbp-yic pants (verify_config_by_default)]$ time for i in `seq 50`;do ./pants goals; done &> /dev/null

real    0m40.051s
user    0m25.151s
sys 0m12.689s

https://travis-ci.org/wisechengyi/pants/builds/119999138

  1. Ship It!
  2. 
      
  1. Ship It!
  2. 
      
Review request changed

Status: Closed (submitted)

Change Summary:

d6eead22e7eebcdfa67dd3098df923f07b105bd6

  1. 
      
  2. I'm not sure what "due to the select few backend packages" means? Is it that there may be config provided for backends that aren't selected?

    1. the plugins are overriden, so e.g. gen.scrooge in pants.ini will not be recognized.

  3. 
      
Loading...