verify config by default

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

Information
Yi Cheng
pants
Reviewers
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

Stu Hood
Mateo Rodriguez
Yi Cheng
Yi Cheng
Review request changed

Status: Closed (submitted)

Change Summary:

d6eead22e7eebcdfa67dd3098df923f07b105bd6

Benjy Weinberger

   

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.

Loading...