Provide more diagnostic info for undefined settings#2009
Conversation
|
Can one of the admins verify this patch? |
|
Nice turnaround :) Perhaps it would be nicer if it were output some like (I hand-wrote this, I mean generally how sbt pretty prints settings): |
|
Is there any existing API for that? |
|
I don't know, but somewhere On 8 May 2015 at 11:14, David Pérez Carmona notifications@github.com
|
|
Yeah, some of the "API" is internal methods that should get turned into API. This is a a really nice improvement! |
|
LGTM, pending some "pretty printing" we may want to try. |
|
macro arg dep nested appears to be flaky. |
|
Would you mind rebasing this so we can get the travis tests passing? I'd like to merge it in. Thanks!
|
|
Sorry for the delay, I've been travelling and quite busy. I've merged the latest SBT modifications on branch 0.13. |
|
Thanks! |
Provide more diagnostic info for undefined settings
|
do we want notes for this? |
|
@dwijnand You can send a PR for a note or leave it. I probably left it alone when I was last checking all the commits. |
Provide more diagnostic info for undefined settings
This solves this issue:
#2008
This is my previous error message:
which doesn't tell me what is happening.
Now, I get this additional information:
Now I know where the problem lies! :-)