Revert "- Update Nodes to use triple slash comments rather than block based comments (this matches Xcode’s documentation style using Editor -> Structure -> Add Documentation)"

This reverts commit 4b8b297.

Author: michaelbrown-info

.github/workflows/artifactory.yml

@@ -5,7 +5,7 @@ on:
     tags: ['[0-9]+.[0-9]+.[0-9]+']
 
 env:
-  DEVELOPER_DIR: /Applications/Xcode_15.4.app/Contents/Developer
+  DEVELOPER_DIR: /Applications/Xcode_15.2.app/Contents/Developer
 
 jobs:
   artifactory:
@@ -18,14 +18,15 @@ jobs:
     - name: Install dependencies
       run: brew install jfrog-cli
     - name: Checkout source
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
     - name: Build documentation
       run: |
         make docs open="no" DERIVED_DATA_PATH="$(mktemp -d)"
         mkdir Documentation
         mv .build/documentation/archive/Nodes.doccarchive Documentation
     - name: Archive
-      run: tar -czf "Nodes-${GITHUB_REF_NAME}.tar.gz" Sources Documentation
+      run: >
+        tar -czf "Nodes-${GITHUB_REF_NAME}.tar.gz" Sources Documentation
     - name: Upload
       run: >
         jf rt u

.github/workflows/docc.yml

@@ -7,14 +7,14 @@ on:
     branches: [ main ]
 
 env:
-  DEVELOPER_DIR: /Applications/Xcode_15.4.app/Contents/Developer
+  DEVELOPER_DIR: /Applications/Xcode_15.2.app/Contents/Developer
 
 jobs:
   docc:
     name: DocC
     runs-on: macos-latest
     steps:
     - name: Checkout source
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
     - name: Build
       run: make docs open="no" DERIVED_DATA_PATH="$(mktemp -d)"

.github/workflows/genesis.yml

@@ -7,7 +7,7 @@ on:
     branches: [ main ]
 
 env:
-  DEVELOPER_DIR: /Applications/Xcode_15.4.app/Contents/Developer
+  DEVELOPER_DIR: /Applications/Xcode_15.2.app/Contents/Developer
 
 jobs:
   genesis:
@@ -21,7 +21,7 @@ jobs:
         brew install mint xcodegen needle mockolo
         mint install yonaskolb/genesis
     - name: Checkout source
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
     - name: Create temporary directory
       id: mktemp
       run: echo "directory=$(mktemp -d)" >> "$GITHUB_OUTPUT"
@@ -41,8 +41,9 @@ jobs:
       run: swift package resolve
     - name: Lint
       working-directory: ${{ steps.mktemp.outputs.directory }}
-      run: >
-        "$(find ".build/artifacts/swiftlint" -type f -name swiftlint -perm +111 -print -quit)"
+      run: |
+        DIRECTORY_NAME=$(echo "${PWD##*/}" | tr '[:upper:]' '[:lower:]')
+        "$(find ".build/artifacts/${DIRECTORY_NAME}" -type f -name swiftlint -perm +111 -print -quit)" \
         lint --strict --reporter github-actions-logging
     - name: Resolve package dependencies
       working-directory: ${{ steps.mktemp.outputs.directory }}
@@ -54,7 +55,7 @@ jobs:
         build-for-testing
         -scheme "Tinder"
         -destination "name=$SIMULATOR,OS=latest"
-        -skipPackagePluginValidation
+        SWIFT_TREAT_WARNINGS_AS_ERRORS=YES
     - name: Test
       working-directory: ${{ steps.mktemp.outputs.directory }}
       run: >

.github/workflows/lint.yml

@@ -7,18 +7,19 @@ on:
     branches: [ main ]
 
 env:
-  DEVELOPER_DIR: /Applications/Xcode_15.4.app/Contents/Developer
+  DEVELOPER_DIR: /Applications/Xcode_15.2.app/Contents/Developer
 
 jobs:
   lint:
     name: Lint
     runs-on: macos-latest
     steps:
     - name: Checkout source
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
     - name: Download swiftlint binary
       run: swift package resolve
     - name: Lint
-      run: >
-        "$(find ".build/artifacts/swiftlint" -type f -name swiftlint -perm +111 -print -quit)"
+      run: |
+        DIRECTORY_NAME=$(echo "${PWD##*/}" | tr '[:upper:]' '[:lower:]')
+        "$(find ".build/artifacts/${DIRECTORY_NAME}" -type f -name swiftlint -perm +111 -print -quit)" \
         lint --strict --reporter github-actions-logging

.github/workflows/pages.yml

@@ -5,7 +5,7 @@ on:
     tags: ['[0-9]+.[0-9]+.[0-9]+']
 
 env:
-  DEVELOPER_DIR: /Applications/Xcode_15.4.app/Contents/Developer
+  DEVELOPER_DIR: /Applications/Xcode_15.2.app/Contents/Developer
 
 jobs:
   pages:
@@ -23,7 +23,7 @@ jobs:
       url: ${{ steps.deployment.outputs.page_url }}
     steps:
     - name: Checkout source
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
     - name: Configure pages
       uses: actions/configure-pages@v2
     - name: Build documentation

.github/workflows/swift.yml

@@ -7,16 +7,39 @@ on:
     branches: [ main ]
 
 env:
-  DEVELOPER_DIR: /Applications/Xcode_15.4.app/Contents/Developer
+  DEVELOPER_DIR: /Applications/Xcode_15.2.app/Contents/Developer
+  SWIFT_STRICT_CONCURRENCY: complete
 
 jobs:
   swift:
     name: Swift
     runs-on: macos-latest
     steps:
     - name: Checkout source
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
+    - name: Validate excluded snapshots
+      run: |
+        SNAPSHOTS=$(find Tests -type d -name "__Snapshots__" | sort)
+        EXCLUDED=$(swift package dump-package | jq -r '.targets[] | select(.type == "test") | "Tests/" + .name + "/" + .exclude[]' | sort)
+        UNEXCLUDED=$(comm -23 <(echo "$SNAPSHOTS") <(echo "$EXCLUDED"))
+        if test -n "$UNEXCLUDED"
+        then
+            echo "::error::Snapshot directories must be excluded from Swift package:" >&2
+            echo "$UNEXCLUDED" >&2
+            exit 1
+        fi
     - name: Build
-      run: swift build -v
+      run: swift build -v -Xswiftc -warnings-as-errors
     - name: Test
       run: swift test -v
+    - name: Delete snapshots
+      run: make delete-snapshots
+    - name: Record snapshots
+      run: swift test -v
+      continue-on-error: true
+    - name: Validate recorded snapshots
+      run: test -z "$(git status --porcelain)"
+    - name: Validate executables
+      run: |
+        swift run -- nodes-code-gen --help
+        swift run -- nodes-xcode-templates-gen --help

.github/workflows/xcode.yml

@@ -7,7 +7,7 @@ on:
     branches: [ main ]
 
 env:
-  DEVELOPER_DIR: /Applications/Xcode_15.4.app/Contents/Developer
+  DEVELOPER_DIR: /Applications/Xcode_15.2.app/Contents/Developer
 
 jobs:
   xcode:
@@ -17,7 +17,7 @@ jobs:
       SIMULATOR: iPhone 14
     steps:
     - name: Checkout source
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
     - name: Resolve package dependencies
       run: xcodebuild -resolvePackageDependencies
     - name: Build
@@ -26,7 +26,7 @@ jobs:
         build-for-testing
         -scheme "Nodes-Package"
         -destination "name=$SIMULATOR,OS=latest"
-        -skipPackagePluginValidation
+        SWIFT_TREAT_WARNINGS_AS_ERRORS=YES
     - name: Test
       run: >
         xcodebuild

.gitignore

@@ -8,3 +8,9 @@
 # Xcode
 *.xcworkspace/
 *.xcodeproj/
+
+# Codegen
+**/Generated/
+
+# Bazel
+bazel-*

.swiftlint-rules.yml

@@ -33,7 +33,6 @@ only_rules:
 - contains_over_filter_is_empty
 - contains_over_first_not_nil
 - contains_over_range_nil_comparison
-# - contrasted_opening_brace
 - control_statement
 - convenience_type
 - custom_rules
@@ -74,7 +73,6 @@ only_rules:
 - file_name
 - file_name_no_space
 - file_types_order
-- final_test_case
 - first_where
 - flatmap_over_map_reduce
 - for_where
@@ -126,13 +124,11 @@ only_rules:
 - multiple_closures_with_trailing_closure
 - nesting
 - nimble_operator
-# - no_empty_block
 - no_extension_access_modifier
 - no_fallthrough_only
 - no_grouping_extension
 # - no_magic_numbers
 - no_space_in_method_call
-- non_optional_string_data_conversion
 - non_overridable_class_declaration
 - notification_center_detachment
 - ns_number_init_as_function_reference
@@ -141,7 +137,6 @@ only_rules:
 - nsobject_prefer_isequal
 - number_separator
 # - object_literal
-# - one_declaration_per_file
 - opening_brace
 - operator_usage_whitespace
 - operator_whitespace
@@ -151,7 +146,6 @@ only_rules:
 - override_in_extension
 - pattern_matching_keywords
 - period_spacing
-- prefer_key_path
 - prefer_nimble
 - prefer_self_in_static_references
 - prefer_self_type_over_type_of_self
@@ -187,7 +181,6 @@ only_rules:
 # - return_value_from_void_function
 - self_binding
 - self_in_property_initialization
-# - shorthand_argument
 - shorthand_operator
 - shorthand_optional_binding
 - single_test_class
@@ -196,7 +189,6 @@ only_rules:
 - sorted_imports
 - statement_position
 - static_operator
-- static_over_final_class
 - strict_fileprivate
 - strong_iboutlet
 - superfluous_disable_command
@@ -229,7 +221,6 @@ only_rules:
 - unused_control_flow_label
 - unused_enumerated
 - unused_optional_binding
-- unused_parameter
 - unused_setter_value
 - valid_ibinspectable
 - vertical_parameter_alignment

.swiftlint.yml

@@ -1,4 +1,5 @@
 included:
+- Plugins
 - Sources
 - Tests