Swiftlint: function_parameter_count should ignore initializers

[ivanbruel]

How would you work around models with a good amount of properties to be set on an initializer?
I agree with the number of parameters being limited, but not initializers themselves.

[jpsim]

Just // swiftlint:disable:next function_parameter_count on the line above the initializer?

[ivanbruel]

It's not about working around but rather trying to understand why it's a best practice when it comes to initializers.

[jpsim]

Ok, thinking about this more I would agree. This was even suggested in the initial issue tracking this rule (https://github.com/realm/SwiftLint/issues/415#issue-128552879). This should be just a matter of removing FunctionConstructor from the functionKinds used to apply the rule: https://github.com/realm/SwiftLint/blob/0.9.0/Source/SwiftLintFramework/Rules/FunctionParameterCountRule.swift#L101

[kaybutter]

I tried applying your fix to the FunctionParameterCountRule but I'm still seeing the warning with my project. The issue seems to be that my constructor is being recognized as FunctionMethodInstance instead of FunctionConstructor. :(

[marcelofabri]

This was fixed in https://github.com/realm/SwiftLint/pull/942