diff --git a/.github/workflows/macOS-tests.yml b/.github/workflows/macOS-tests.yml index db570bbf0..3e3bafb6f 100644 --- a/.github/workflows/macOS-tests.yml +++ b/.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 diff --git a/.swiftlint.yml b/.swiftlint.yml index a794f4992..f7519e42f 100644 --- a/.swiftlint.yml +++ b/.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.