Correctly provide EditorConfig property types for Ktlint

[Goooler]

• Closes KtLint + Gradle: java.lang.ClassCastException: class java.lang.String cannot be cast to class java.lang.Integer exception thrown when .editorconfig file is present #1904.
• Refs ktlint_function_signature_rule_force_multiline_when_parameter_count_greater_or_equal_than values can't be correctly read pinterest/ktlint#2560 (comment).

[paul-dingemans]

@Goooler It is implemented as documented and intended from the Ktlint perspective. This might also fix problems with .editorconfig properties below:

• ktlint_argument_list_wrapping_ignore_when_parameter_count_greater_or_equal_than
• ktlint_chain_method_rule_force_multiline_when_chain_operator_count_greater_or_equal_than
• ktlint_class_signature_rule_force_multiline_when_parameter_count_greater_or_equal_than
• ktlint_function_signature_rule_force_multiline_when_parameter_count_greater_or_equal_than

[Goooler]

Do you mean I can add both of them into the test fixture?

[paul-dingemans]

Do you mean I can add both of them into the test fixture?

What do you mean with 'both'? I have listed 4 properties which would result in an exception when set in .editorconfig with old spotless version.

[Goooler]

Yeah, I meant these 4 rules. Would you suggest I add them into the test fixture?

[paul-dingemans]

Yeah, I meant these 4 rules. Would you suggest I add them into the test fixture?

No, that would not make sense to me. You will never be able to keep that list in sync with Ktlint whenever it would add a new property which has a non-string datatype. I just mentioned the other properties, so that it can be used in documentation, or to verify whether there are other (open) issues which are related.

[Goooler]

Got it! Thanks again for your patience!

[zengcanxiang]

@nedtwigg hi, In which version will this MR change be released?

[henningBunk]

@zengcanxiang I got errors with 6.25.0 but it works with 7.0.0.BETA1