[CIS-2245] Implement Fastlane Stream Actions (#230)

Author: testableapple

.github/workflows/cron-checks.yml

@@ -75,3 +75,12 @@ jobs:
         SLACK_WEBHOOK_URL: ${{ secrets.SLACK_WEBHOOK_URL }}
         MATRIX_CONTEXT: ${{ toJson(matrix) }}
       if: failure()
+
+  automated-code-review:
+    name: Automated Code Review
+    runs-on: macos-12
+    steps:
+    - uses: actions/[email protected]
+    - uses: ./.github/actions/bootstrap
+    - run: bundle exec fastlane rubocop
+    - run: ./Scripts/run-linter.sh

.github/workflows/smoke-checks.yml

@@ -27,14 +27,17 @@ jobs:
   automated-code-review:
     name: Automated Code Review
     runs-on: macos-12
+    env:
+      GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+      GITHUB_PR_NUM: ${{ github.event.number }}
     if: ${{ github.event_name != 'push' }}
     steps:
     - uses: actions/[email protected]
     - uses: ./.github/actions/bootstrap
     - name: Run Danger
       run: bundle exec danger
-      env:
-        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+    - name: Run Fastlane Linting
+      run: bundle exec fastlane rubocop
     - name: Run Linting
       run: ./Scripts/run-linter.sh
 
@@ -56,7 +59,7 @@ jobs:
       timeout-minutes: 30
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        GITHUB_EVENT: ${{ toJson(github.event) }}
+        GITHUB_PR_NUM: ${{ github.event.number }}
     - name: Get branch name
       id: get_branch_name
       run: echo "branch=${GITHUB_REF#refs/heads/}" >> $GITHUB_OUTPUT
@@ -112,7 +115,7 @@ jobs:
       run: bundle exec fastlane test_e2e_mock device:"${{ env.IOS_SIMULATOR_DEVICE }}" batch:'${{ matrix.batch }}'
       env:
         GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        GITHUB_EVENT: ${{ toJson(github.event) }}
+        GITHUB_PR_NUM: ${{ github.event.number }}
         MATRIX_SIZE: ${{ strategy.job-total }}
     - name: Allure TestOps Upload
       if: env.LAUNCH_ID != '' && (success() || failure())

Dangerfile

@@ -1,22 +1,22 @@
-skip_danger_check = github.pr_body.include? "#skip_danger"
+skip_danger_check = github.pr_body.include?("#skip_danger")
 
 if skip_danger_check
-    message("Skipping Danger due to skip_danger tag")
-    return
+  message("Skipping Danger due to skip_danger tag")
+  return
 end
 
 # Make it more obvious that a PR is a work in progress and shouldn't be merged yet.
 if github.pr_json["mergeable_state"] == "draft"
-    message("Skipping Danger since PR is classed as Draft")
-    return
+  message("Skipping Danger since PR is classed as Draft")
+  return
 end
 
 # Warn when there is a big PR.
 warn("Big PR") if git.lines_of_code > 500
 
 # Mainly to encourage writing up some reasoning about the PR, rather than just leaving a title.
 if github.pr_body.length < 3 && git.lines_of_code > 50
-    warn("Please provide a summary in the Pull Request description")
+  warn("Please provide a summary in the Pull Request description")
 end
 
 ## Let's check if there are any changes in the project folder
@@ -29,18 +29,18 @@ has_app_changes = !git.modified_files.grep(/Sources/).empty?
 #     warn("Please adds tests!")
 # end
 
-has_meta_label = github.pr_labels.any? { |label| label.include? "meta" }
-has_demo_label = github.pr_labels.any? { |label| label.include? "demo" }
-has_no_changelog_tag = github.pr_body.include? "#no_changelog"
-has_skip_changelog_tag = github.pr_body.include? "#skip_changelog"
+has_meta_label = github.pr_labels.any? { |label| label.include?("meta") }
+has_demo_label = github.pr_labels.any? { |label| label.include?("demo") }
+has_no_changelog_tag = github.pr_body.include?("#no_changelog")
+has_skip_changelog_tag = github.pr_body.include?("#skip_changelog")
 
 has_changelog_escape = has_meta_label || has_demo_label || has_no_changelog_tag || has_skip_changelog_tag
 
 # Add a CHANGELOG entry for app changes
 if !has_changelog_escape && !git.modified_files.include?("CHANGELOG.md") && has_app_changes
-    message("There seems to be app changes but CHANGELOG wasn't modified.\nPlease include an entry if the PR includes user-facing changes.\nYou can find it at [CHANGELOG.md](https://github.com/GetStream/stream-chat-swift/blob/main/CHANGELOG.md).")
+  message("There seems to be app changes but CHANGELOG wasn't modified.\nPlease include an entry if the PR includes user-facing changes.\nYou can find it at [CHANGELOG.md](https://github.com/GetStream/stream-chat-swift/blob/main/CHANGELOG.md).")
 end
 
 # Check all commits have correct format. Disable the length rule, since it's hardcoded
 # to 50 and GitHub has the limit 80.
-commit_lint.check disable: [:subject_length]
+commit_lint.check(disable: [:subject_length])

Gemfile

@@ -2,7 +2,7 @@
 
 source "https://rubygems.org"
 
-git_source(:github) {|repo_name| "https://github.com/#{repo_name}" }
+git_source(:github) { |repo_name| "https://github.com/#{repo_name}" }
 
 gem 'cocoapods'
 gem 'danger'
@@ -12,7 +12,7 @@ gem 'fastlane-plugin-lizard'
 gem 'jazzy'
 gem 'json'
 gem 'plist'
-gem 'rubocop', '1.12.1'
+gem 'rubocop', '1.38'
 gem 'rubocop-performance'
 gem 'rubocop-require_tools'
 gem 'sinatra'