Merge pull request #784 from cclauss/patch-1

SwiftLint recommends a double run when fixing

Author: JeneaVranceanu

.github/workflows/macOS-tests.yml

@@ -41,8 +41,10 @@ jobs:
           codespell  # See .codespellrc for args
       - name: SwiftLint
         run: |
-          # Adding --fix flag makes CI print only errors that cannot be fixed automatically
-          swiftlint --fix
+          # 1. Make all automated fixes that are possible
+          # 2. git diff to see what (if any) automated fixes were made
+          # 3. See https://github.com/realm/SwiftLint#xcode explains why the double run
+          swiftlint --fix --quiet && git diff && swiftlint --quiet
       - name: Resolve dependencies
         run: swift package resolve
       - name: Build

.swiftlint.yml

@@ -60,10 +60,6 @@ opt_in_rules:
   - unneeded_parentheses_in_closure_argument
   - weak_delegate
 
-# force warnings
-force_cast: error
-force_try: error
-
 custom_rules:
   commented_out_code:
     included: .*\.swift # regex that defines paths to include during linting. optional.