Give a hint on how to disable the Invalid config entries detected message

Review Request #3642 — Created April 2, 2016 and submitted

zundel
pants
3134
edaa7f1...
pants-reviews
gmalmquist, wisechengyi

Suggest --no-verify-config in the exception message

CI run: https://travis-ci.org/pantsbuild/pants/builds/120279298

In our repo:

...
ERROR] Invalid option 'python_test_paths' under [idea] in /Users/zundel/Development/java/pants.ini
Exception caught: (<class 'pants.option.errors.OptionsError'>)

Exception message: Invalid config entries detected. See log for entries to remove from pants.ini files.
(Specify --no-verify-config to disable this check.)
  1. Rather than recommending this, could we instead suggest that people register the options they have in pants.ini?

  2. 
      
  1. ...or just add a hint that that is possible at least?

    1. Sounds good. There are legitimate reasons for not wanting it on, if you are testing multiple versions of pants in the same repo, but I agree that is the main thing you should do!

  2. 
      
  1. 
      
  2. pants.ini files may sound too specific, and the error log above has already indicated where the invalid entires are.

    Maybe Invalid config entries detected. See log for details. Please either correct the entries in config file(s) or disable the check with --no-verify-config.

  3. 
      
Review request changed

Status: Closed (submitted)

Change Summary:

Thanks all commit 3924cfc

Loading...