Pass down error log about danger-periphery HOT 4 OPEN

The error logs of periphery is written in your log. Isn't it enough?

Shell command '/usr/bin/env xcodebuild -project /Users/test/Some.xcodeproj -list -json' returned exit status '74'

This error is from here https://github.com/peripheryapp/periphery/blob/2173f0d5cb09bf5708776224b0b56f239dbff3f5/Sources/XcodeSupport/Xcodebuild.swift#L76

from danger-periphery.

@manicmaniac Not really as exit code 74 is really broad. Seems like we don't spit out the stdout here(

when the exception is raised so all the detailed logs is never printed out.

Also this is possibly a separate issue but sending down verbose: true is not respected as I don't see the verbose periphery logs either

from danger-periphery.

Hmm... let me get this straight.

Shell command '/usr/bin/env xcodebuild -project /Users/test/Some.xcodeproj -list -json' returned exit status '74'

This message includes all output from stderr. I agree exit status '74' is so broad but that is the raw text from Periphery.
The former part xcodebuild -project /Users/test/Some.xcodeproj -list -json looks more important in this case.

when the exception is raised so all the detailed logs is never printed out.

As you say danger-periphery suppresses stdout once an error occurs.
However, about this case, I don't think analyzing stdout helps your debug because Periphery dies in early stage and I guess it writes nothing meaningful to stdout.

Also this is possibly a separate issue but sending down verbose: true is not respected as I don't see the verbose periphery logs either

Yes, passing verbose: true option to periphery.scan is passed through to Periphery executable so danger-periphery doesn't recognize it. In other words, there's no "verbose" option in danger-periphery now.

Generally speaking, I agree that danger-periphery might have to print stdout from Periphery for more complex error. But ususally stdout log is much bigger than stderr log so simply adding stdout log to raised error doesn't sound nice.

If you have a good idea, I'm welcome to hear that or you can open a pull request.

from danger-periphery.

"Yes, passing verbose: true option to periphery.scan is passed through to Periphery executable so danger-periphery doesn't recognize it. In other words, there's no "verbose" option in danger-periphery now."

It would be nice to be able to override this. I'm trying to debug an issue right now where I see no output at all on my CI machines, while running danger locally detects one violation.

It's hard to tell if periphery is even running on the CI machines.

from danger-periphery.