Skip to content

[Connect] Fix API inconsistent naming + Add Dashboard-only support for Account Management component #67

[Connect] Fix API inconsistent naming + Add Dashboard-only support for Account Management component

[Connect] Fix API inconsistent naming + Add Dashboard-only support for Account Management component #67

Workflow file for this run

name: Dead code detection
on:
pull_request:
types: [opened, labeled, unlabeled, synchronize]
paths:
- '**/*.swift'
jobs:
dead-code-check:
runs-on: macos-13
permissions:
pull-requests: write
steps:
- name: Checkout code
uses: actions/checkout@v2
with:
fetch-depth: 0
- name: Install Periphery
run: brew install peripheryapp/periphery/periphery
- name: Build project and run Periphery scan
id: periphery-scan
run: |
periphery scan --config .periphery.yml --clean-build 2>&1 | sed -E 's#.*/##; s/:[0-9]+:[0-9]+:/: /' | grep 'is unused' > periphery_report_feature_formatted_sorted.txt
ruby ci_scripts/dead_code/process_periphery_output.rb periphery_report_feature_formatted_sorted.txt unused_code_feature.json
- name: Copy .periphery.yml to temporary location
run: |
# Copy necessary files to /tmp/ before checking out master
cp .periphery.yml /tmp/
cp ci_scripts/dead_code/process_periphery_output.rb /tmp/
- name: Compare Periphery output with master baseline
run: |
git fetch origin master:master
git checkout master
cp /tmp/.periphery.yml .periphery.yml
mkdir -p ci_scripts/dead_code/
cp /tmp/process_periphery_output.rb ci_scripts/dead_code/
periphery scan --config .periphery.yml --clean-build 2>&1 | sed -E 's#.*/##; s/:[0-9]+:[0-9]+:/: /' | grep 'is unused' > periphery_report_master_formatted_sorted.txt
ruby ci_scripts/dead_code/process_periphery_output.rb periphery_report_master_formatted_sorted.txt unused_code_master.json
- name: Compare Unused Code JSON Files
id: compare-dead-code
run: |
# Compare the keys in the JSON files to find new dead code
ruby -r json -e '
master_file = "unused_code_master.json"
feature_file = "unused_code_feature.json"
output_file = "new_dead_code.json"
master_unused_code = JSON.parse(File.read(master_file))
feature_unused_code = JSON.parse(File.read(feature_file))
new_dead_code = feature_unused_code.reject { |k, _| master_unused_code.key?(k) }
if new_dead_code.size > 200
puts "More than 200 keys present, skipping. This usually happens if a build fails"
elsif new_dead_code.empty?
puts "No new dead code detected."
else
File.write(output_file, JSON.pretty_generate(new_dead_code) + "\n")
end
'
# Check if new_dead_code.json exists and is not empty
if [ -s new_dead_code.json ]; then
echo "New dead code detected."
echo "diff<<EOF" >> $GITHUB_ENV
cat new_dead_code.json >> $GITHUB_ENV
echo "EOF" >> $GITHUB_ENV
else
echo "No new dead code detected."
fi
- uses: peter-evans/find-comment@v3
id: find_comment
with:
issue-number: ${{ github.event.pull_request.number }}
body-includes: '🚨 New dead code detected'
- uses: peter-evans/create-or-update-comment@v3
id: create_update_comment
if: env.diff != ''
with:
issue-number: ${{ github.event.pull_request.number }}
body: |
🚨 New dead code detected in this PR:
```diff
${{ env.diff }}
```
Please remove the dead code before merging.
If this is intentional, you can bypass this check by adding the label `skip dead code check` to this PR.
ℹ️ If this comment appears to be left in error, make sure your branch is up-to-date with `master`.
edit-mode: replace
comment-id: ${{ steps.find_comment.outputs.comment-id }}
token: ${{ secrets.GITHUB_TOKEN }}
- name: Fail if not acknowledged
if: env.diff != '' && !contains(github.event.pull_request.labels.*.name, 'skip dead code check')
run: exit 1