Skip to content

Commit ae8ae58

Browse files
HweinstockHweinstock
committed
put input in env before passing
1 parent 0536f09 commit ae8ae58

File tree

1 file changed

+76
-74
lines changed

1 file changed

+76
-74
lines changed

‎.github/workflows/copyPasteDetection.yml‎

Lines changed: 76 additions & 74 deletions
Original file line numberDiff line numberDiff line change
@@ -1,86 +1,88 @@
1-
# # github actions: https://docs.github.com/en/actions/use-cases-and-examples/building-and-testing/building-and-testing-nodejs
2-
# # setup-node: https://github.com/actions/setup-node
1+
# github actions: https://docs.github.com/en/actions/use-cases-and-examples/building-and-testing/building-and-testing-nodejs
2+
# setup-node: https://github.com/actions/setup-node
33

4-
# name: Copy-Paste Detection
4+
name: Copy-Paste Detection
55

6-
# on:
7-
# pull_request:
8-
# branches: [master, feature/*, staging]
6+
on:
7+
pull_request:
8+
branches: [master, feature/*, staging]
99

10-
# jobs:
11-
# jscpd:
12-
# runs-on: ubuntu-latest
13-
# strategy:
14-
# matrix:
15-
# node-version: [18.x]
16-
# env:
17-
# NODE_OPTIONS: '--max-old-space-size=8192'
10+
jobs:
11+
jscpd:
12+
runs-on: ubuntu-latest
13+
strategy:
14+
matrix:
15+
node-version: [18.x]
16+
env:
17+
NODE_OPTIONS: '--max-old-space-size=8192'
1818

19-
# steps:
20-
# - uses: actions/checkout@v4
21-
# with:
22-
# fetch-depth: 0
19+
steps:
20+
- uses: actions/checkout@v4
21+
with:
22+
fetch-depth: 0
2323

24-
# - name: Use Node.js ${{ matrix.node-version }}
25-
# uses: actions/setup-node@v4
26-
# with:
27-
# node-version: ${{ matrix.node-version }}
24+
- name: Use Node.js ${{ matrix.node-version }}
25+
uses: actions/setup-node@v4
26+
with:
27+
node-version: ${{ matrix.node-version }}
2828

29-
# - name: Fetch fork upstream
30-
# run: |
31-
# git remote add forkUpstream https://github.com/${{ github.event.pull_request.head.repo.full_name }} # URL of the fork
32-
# git fetch forkUpstream # Fetch fork
29+
- name: Fetch fork upstream
30+
env:
31+
REPO_NAME: ${{ github.event.pull_request.head.repo.full_name }}
32+
run: |
33+
git remote add forkUpstream https://github.com/$REPO_NAME # URL of the fork
34+
git fetch forkUpstream # Fetch fork
3335
34-
# - name: Determine base and target branches for comparison.
35-
# run: |
36-
# echo "CURRENT_BRANCH=${{ github.head_ref }}" >> $GITHUB_ENV
37-
# echo "TARGET_BRANCH=${{ github.event.pull_request.base.ref }}" >> $GITHUB_ENV
38-
# - run: git diff --name-only origin/$TARGET_BRANCH forkUpstream/$CURRENT_BRANCH > diff_output.txt
39-
# - run: |
40-
# npm install -g jscpd
36+
- name: Compute git diff
37+
env:
38+
CURRENT_BRANCH: ${{ github.head_ref }}
39+
TARGET_BRANCH: ${{ github.event.pull_request.base.ref }}
40+
run: git diff --name-only origin/$TARGET_BRANCH forkUpstream/$CURRENT_BRANCH > diff_output.txt
4141

42-
# - run: jscpd --config "$GITHUB_WORKSPACE/.github/workflows/jscpd.json"
42+
- run: npm install -g jscpd
4343

44-
# - if: always()
45-
# uses: actions/upload-artifact@v4
46-
# with:
47-
# name: unfiltered-jscpd-report
48-
# path: ./jscpd-report.json
44+
- run: jscpd --config "$GITHUB_WORKSPACE/.github/workflows/jscpd.json"
4945

50-
# - name: Filter jscpd report for changed files
51-
# run: |
52-
# if [ ! -f ./jscpd-report.json ]; then
53-
# echo "jscpd-report.json not found"
54-
# exit 1
55-
# fi
56-
# echo "Filtering jscpd report for changed files..."
57-
# CHANGED_FILES=$(jq -R -s -c 'split("\n")[:-1]' diff_output.txt)
58-
# echo "Changed files: $CHANGED_FILES"
59-
# jq --argjson changed_files "$CHANGED_FILES" '
60-
# .duplicates | map(select(
61-
# (.firstFile?.name as $fname | $changed_files | any(. == $fname)) or
62-
# (.secondFile?.name as $sname | $changed_files | any(. == $sname))
63-
# ))
64-
# ' ./jscpd-report.json > filtered-jscpd-report.json
65-
# cat filtered-jscpd-report.json
46+
- if: always()
47+
uses: actions/upload-artifact@v4
48+
with:
49+
name: unfiltered-jscpd-report
50+
path: ./jscpd-report.json
6651

67-
# - name: Check for duplicates
68-
# run: |
69-
# if [ $(wc -l < ./filtered-jscpd-report.json) -gt 1 ]; then
70-
# echo "filtered_report_exists=true" >> $GITHUB_ENV
71-
# else
72-
# echo "filtered_report_exists=false" >> $GITHUB_ENV
73-
# fi
74-
# - name: upload filtered report (if applicable)
75-
# if: env.filtered_report_exists == 'true'
76-
# uses: actions/upload-artifact@v4
77-
# with:
78-
# name: filtered-jscpd-report
79-
# path: ./filtered-jscpd-report.json
52+
- name: Filter jscpd report for changed files
53+
run: |
54+
if [ ! -f ./jscpd-report.json ]; then
55+
echo "jscpd-report.json not found"
56+
exit 1
57+
fi
58+
echo "Filtering jscpd report for changed files..."
59+
CHANGED_FILES=$(jq -R -s -c 'split("\n")[:-1]' diff_output.txt)
60+
echo "Changed files: $CHANGED_FILES"
61+
jq --argjson changed_files "$CHANGED_FILES" '
62+
.duplicates | map(select(
63+
(.firstFile?.name as $fname | $changed_files | any(. == $fname)) or
64+
(.secondFile?.name as $sname | $changed_files | any(. == $sname))
65+
))
66+
' ./jscpd-report.json > filtered-jscpd-report.json
67+
cat filtered-jscpd-report.json
8068
81-
# - name: Fail and log found duplicates.
82-
# if: env.filtered_report_exists == 'true'
83-
# run: |
84-
# cat ./filtered-jscpd-report.json
85-
# echo "Duplications found, failing the check."
86-
# exit 1
69+
- name: Check for duplicates
70+
run: |
71+
if [ $(wc -l < ./filtered-jscpd-report.json) -gt 1 ]; then
72+
echo "filtered_report_exists=true" >> $GITHUB_ENV
73+
else
74+
echo "filtered_report_exists=false" >> $GITHUB_ENV
75+
fi
76+
- name: upload filtered report (if applicable)
77+
if: env.filtered_report_exists == 'true'
78+
uses: actions/upload-artifact@v4
79+
with:
80+
name: filtered-jscpd-report
81+
path: ./filtered-jscpd-report.json
82+
83+
- name: Fail and log found duplicates.
84+
if: env.filtered_report_exists == 'true'
85+
run: |
86+
cat ./filtered-jscpd-report.json
87+
echo "Duplications found, failing the check."
88+
exit 1

0 commit comments

Comments
 (0)