I don't use neither IBAnalyzer nor SwiftLint but I guess it would be marvellous to be able to lint for something like this. It might not make sense though so feel free to close.

I Installed IBAnalyzer from cocoapods and it is working oppositely.

Example: Added line @IBOutlet private weak var someLabel: UILabel!

There is no any warning. Connected UILabel to this IBOutlet And appeared warning /Users/.../XcodeProjects/ScriptsTest/ScriptsTest/Base.lproj/Main.storyboard:37:94: warning: IBOutlet missing: someLabel is not implemented in Main.storyboard

But I compiled sources and replaced IBAnalyzer.app in cocoapods folder and it works correctly.

Xcode9.2 fails to build the project because of SourceKittenFramework. Like:

Clang+SourceKitten.swift:81:52: error: cannot use optional chaining on non-optional value of type 'String'
        return rootXML["Declaration"].element?.text?

Changes:

• fixed build errors;
• fixed swiftlint warnings;
• updated project to use recommended settings;
• updated project's dependencies to the latest versions.

Building the project from scratch to be able to use it is cumbersome. We should come up with a better way to distribute it, e.g. by building a binary the way SwiftGen does.

I installed this in ~/bin so I could access it from any project. However, when launching it, it complains that it cannot find the file.

e.g.

ibanalyzer MyProject results in Path /Users/troy/bin/IBAnalyzer.app/Contents/MyProject doesn't exist.

However, launching it like ibanalyzer ~/Projects/MyProject works.

The scripts in the README are now wrapped in quotes and the PODS_ROOT environmental variable is wrapped with curly brackets as that is the normal way of using it.

This makes them more robust for spaces in paths and CI etc.

The current way of distribution is obviously too cumbersome. To be able to easily distribute IBAnalyzer through CocoaPods or Homebrew we have to be able to build a single binary containing all required dynamic frameworks.

I didn't look into that yet but I think that Sourcery and SwiftGen have this already figured out.

Just a suggestion - anyway that this could be turned into a run script when you build or run your app? It'll throw a warning/error if it finds an issue. Alternatively, SwiftLint integration seems appropriate, since these types of issues are akin to their force_try rule, which throws an error if you implicitly unwrap any optionals.

Just my $0.02.

Thanks for your work!

Environment

XCode 9.3 Swift 4.1

Problem

IBAnalyzer gives and an error xib:19:88: warning: IBOutlet missing: tableHeight is not implemented in SomeView.xib even though tableHeight is implemented. There's no such error with XCode 9.2 and Swift 4.0.3.

Solution

The root of the problem is incorrect parsing, which results in Class with empty outlets/actions. Need to fix code parsing for Swift 4.1.

Hi @fastred .

Does this approach make sense to you all?

exit 0 No errors, maybe warnings in non-strict mode exit 1 Usage or system error exit 2 Style violations of severity "error" exit 3 No style violations of severity "error", but violations of severity "warning" with strict

I don't want IBAnalyzer to parse every folder and file in my project folder. I need to maintain an ignore list, this will make my analysing more faster. Is there any support for this?

Hey, I finally got some time to test IBAnalyzer in some projects I'm working on. It shows some nice results :-) But I also miss some features like ignoring files or specifying multiple paths for analyzing.

As there are no commits in the last months: Are you still have plans to continue the development?

So we have some xibs where we have a UIViewController and another object in the xib (you can do this by dragging in an Object from the Object Library) and the UIViewController has some buttons and we link the actions from the buttons to the Object. This causes IBAnalyzer to show warnings because the action isn't linked to the UIViewController.

/.../.../UI/Search/SearchStoryboard.storyboard:53:106: warning: IBAction missing: cancelButton: is not implemented in SearchStoryboard.storyboard

I saw this output and am thinking it can be improved by elaborate the view controller name or id. Something like:

/.../.../UI/Search/SearchStoryboard.storyboard:53:106: warning: IBAction missing: cancelButton: is not implemented in SearchStoryboard.storyboard in Sub Genre View Controller Scene

Thanks.

There are some issues that we won't be able to address, e.g. when a superclass's source isn't available but it already defines outlets or actions (see #21). For that case we should allow a user to pass a list of types that we should simply ignore, e.g.:

ibanalyzer --ignored="MyView,SomeViewController ./