Remove some settings causing compilation errors and reduce other annoying warnings

[jeden]

Related Issue

N/A

Summary of Changes

Removed settings causing compilation errors and reduced other annoying warnings

Need Regression Testing

• Yes
• No

Risk Assessment

• Low
• Medium
• High

Additional Notes

This PR does not affect the app from a functional standpoint, it's all about the Swift Linter

Screenshots (if applicable)

N/A

[lmcmz]

I don't think disable those rules are good practices

[jeden]

I don't think disable those rules are good practices

The first 5 cause compilation to fail in the current code, so keeping them means refactoring.

• no_space_in_method_call causes warnings reported on cases where a function takes a closure and it's called without parenthesis, for example: DispatchQueue.main.async {
• line_length I personally tend to ignore it, as in some cases I prefer a longer line to splits
• switch_case_alignment seems to randomly throw warnings even if cases are aligned correctly
• syntactic_sugar: I find this annoying, as sometimes I prefer to make the code more explicit by avoiding syntactic sugar — typically in some usages of optionals, or when defining extensions.

Anyway it seems nobody but me has the linter enabled, as it causes compilation to fail.