fastred / IBAnalyzer

Find common xib and storyboard-related problems without running your app or writing unit tests.

http://holko.pl

MIT License

951 stars 26 forks source link

Support Xcode report integration #23

Closed elneruda closed 7 years ago

elneruda commented 7 years ago

Implement xcode report style : show warnings with line & column

9 Suport multifile IBAction/IBOutlet extension

fastred commented 7 years ago

Thank you ❤️. This looks really nice! I'm going to take a closer look in a few days, when I have more time.

One significant suggestion that I have right now is to not include bin/IBAnalyzer.app in the repo. It's pretty large and would have to be updated with each release.

For now we could point people to Releases tab which contains prebuilt binaries. Later on I'm planning to distribute binary version through CocoaPods, similarly to SwiftGen, and through Homebrew.

elneruda commented 7 years ago

@fastred great! I've just improved severity (warning/error), will add it in a new pull-request noted for IBAnalyzer.app: will be removed from repo

fastred commented 7 years ago

Cool. Would be great if you could remove IBAnalyzer.app in this PR. I'd squash it before merging, so that it wouldn't increase the repo's size.

elneruda commented 7 years ago

@fastred I've just removed IBPAnalyzer.app

elneruda commented 7 years ago

@fastred i've just pushed requested improvements

fastred commented 7 years ago

Thanks! I'll get the CocoaPods integration sorted out soon.

elneruda commented 7 years ago

great!!