Integrate swiftformat to infra

.swift-version

@@ -0,0 +1 @@
+5.9

.swiftformat

@@ -0,0 +1,29 @@
+# https://github.com/nicklockwood/SwiftFormat/blob/main/Rules.md
+--exclude Sources/ShellParserGenerated
+--indentcase true
+--patternlet inline
+--indentstrings true
+
+# https://github.com/nicklockwood/SwiftFormat/issues/483 fix indentation for expressions nested in ternary
+--wrapternary before-operators
+
+--disable andOperator
+--disable blankLinesBetweenScopes
+--disable consecutiveSpaces
+--disable consistentSwitchCaseSpacing
+--disable hoistTry
+--disable preferKeyPath
+--disable redundantNilInit
+--disable redundantParens
+--disable redundantRawValues
+--disable redundantReturn
+--disable redundantSelf
+--disable redundantStaticSelf
+--disable redundantType
+--disable spaceAroundComments
+--disable spaceInsideComments
+--disable wrapArguments
+--disable wrapMultilineConditionalAssignment
+
+# This rule is cool but buggy. It feels like heuristics are used instead of real code analysis
+--disable unusedArguments

.swiftlint.yml

@@ -29,7 +29,6 @@ only_rules:
   - no_fallthrough_only
   - no_space_in_method_call
   - nsobject_prefer_isequal # NSObject subclasses should implement isEqual instead of ==
-  - opening_brace
   - operator_whitespace
   - optional_enum_case_matching
   - redundant_discardable_let

Package.swift

@@ -60,7 +60,7 @@ let package = Package(
         .executableTarget(
             name: "AeroSpaceApp",
             dependencies: [
-                .target(name: "AppBundle")
+                .target(name: "AppBundle"),
             ]
         ),
         .executableTarget(

run-tests.sh

@@ -11,5 +11,8 @@ swift test
 ./script/install-deps.sh --swiftlint
 ./swift-exec-deps/.build/debug/swiftlint lint --quiet
 
+./script/install-deps.sh --swiftformat
+./swift-exec-deps/.build/debug/swiftformat .
+
 ./generate.sh --all
 ./script/check-uncommitted-files.sh

script/install-deps.sh

@@ -6,6 +6,7 @@ all=1
 antlr=0
 complgen=0
 swiftlint=0
+swiftformat=0
 xcodegen=0
 while [[ $# -gt 0 ]]; do
     case $1 in
@@ -29,6 +30,11 @@ while [[ $# -gt 0 ]]; do
             xcodegen=1
             shift
             ;;
+        --swiftformat)
+            all=0
+            swiftformat=1
+            shift
+            ;;
         *)
             echo "Unknown option $1"
             exit 1
@@ -70,3 +76,9 @@ if test $all == 1 || test $xcodegen == 1; then
         swift run --package-path ./swift-exec-deps xcodegen --version
     fi
 fi
+
+if test $all == 1 || test $swiftformat == 1; then
+    if ! check-version 0.54.4 ./swift-exec-deps/.build/debug/swiftformat --version; then
+        swift run --package-path ./swift-exec-deps swiftformat --version
+    fi
+fi

swift-exec-deps/Package.resolved

@@ -99,6 +99,15 @@
         "version" : "6.0.3"
       }
     },
+    {
+      "identity" : "swiftformat",
+      "kind" : "remoteSourceControl",
+      "location" : "https://github.com/nicklockwood/SwiftFormat",
+      "state" : {
+        "revision" : "c6680dd33c013abdd18266538e302f6323fa130e",
+        "version" : "0.54.4"
+      }
+    },
     {
       "identity" : "swiftlint",
       "kind" : "remoteSourceControl",

swift-exec-deps/Package.swift

@@ -10,6 +10,7 @@ let package = Package(
     dependencies: [
         .package(url: "https://github.com/yonaskolb/XcodeGen.git", exact: "2.42.0"),
         .package(url: "https://github.com/realm/SwiftLint", exact: "0.56.2"),
+        .package(url: "https://github.com/nicklockwood/SwiftFormat", exact: "0.54.4"),
     ],
     targets: []
 )