marklarr / mayday

Easily add custom warnings and errors to your Xcode project's build process
MIT License
260 stars 7 forks source link

Bribe SuperMarin to show Mayday output when used with xcpretty #3

Closed kylef closed 10 years ago

kylef commented 10 years ago
$ xcodebuild -workspace NSAttributedString+CCLFormat.xcworkspace -scheme Tests test | xcpretty -c
▸ Building Pods/Pods-NSAttributedString+CCLFormat [Debug]
▸ Building Pods/Pods [Debug]
▸ Building NSAttributedString+CCLFormat/Tests [Debug]
▸ Running script 'Check Pods Manifest.lock'
▸ Running script 'Copy Pods Resources'
▸ Running script 'Generate Mayday Flags'
All tests
Test Suite Tests.xctest started
NSAttributedStringTests
    ✓ testAttributedFormatString (0.007 seconds)
    ✓ testAttributedFormatStringCanFormatAttributedString (0.000 seconds)
    ✓ testAttributedFormatStringCanFormatDescription (0.000 seconds)
    ✓ testAttributedFormatStringCanFormatString (0.000 seconds)

     Executed 4 tests, with 0 failures (0 unexpected) in 0.007 (0.009) seconds

Without xcpretty I can see the following:

/Users/kylef/Projects/cocode/NSAttributedString-CCLFormat/Tests/NSAttributedString+CCLFormatTests.m:25: warning: Length of line 124 is longer than 120 characters! [Wmayday]
/Users/kylef/Projects/cocode/NSAttributedString-CCLFormat/Tests/NSAttributedString+CCLFormatTests.m:56: warning: Length of line 147 is longer than 120 characters! [Wmayday]
/Users/kylef/Projects/cocode/NSAttributedString-CCLFormat/Tests/NSAttributedString+CCLFormatTests.m:59: warning: Length of line 145 is longer than 120 characters! [Wmayday]
/Users/kylef/Projects/cocode/NSAttributedString-CCLFormat/Tests/NSAttributedString+CCLFormatTests.m:74: warning: Length of line 126 is longer than 120 characters! [Wmayday]
marklarr commented 10 years ago

+1

kylef commented 10 years ago

/cc @supermarin

supermarin commented 10 years ago

This is actually related to xcpretty not showing warnings. I'm finally getting back on track, will definitely tackle this.

Please if you don't mind, close this one and open an issue on xcpretty for showing warnings.

marklarr commented 10 years ago

Sounds good @supermarin!