Skip to content

Commit 43a875d

Browse files
committed
For new CI, fixed paths for copying of plots when there's a difference in output from PR changes
1 parent 9acd120 commit 43a875d

File tree

2 files changed

+6
-7
lines changed

2 files changed

+6
-7
lines changed

.github/workflows/test-pr.yml

+2-3
Original file line numberDiff line numberDiff line change
@@ -200,7 +200,7 @@ jobs:
200200
if: ${{ always() }}
201201
run: |
202202
mkdir -p $GITHUB_WORKSPACE/test_report
203-
cp -r $GITHUB_WORKSPACE/candidate/build/Run/output_${{ matrix.configuration }}/* $GITHUB_WORKSPACE/test_report/
203+
cp -r $GITHUB_WORKSPACE/candidate/build/Run/output_diff/diff_plots/* $GITHUB_WORKSPACE/test_report/
204204
205205
- name: Attach diff plots to PR
206206
if: ${{ always() }}
@@ -214,5 +214,4 @@ jobs:
214214
with:
215215
name: test-reports
216216
path: |
217-
${{ github.workspace }}/test_report/**.html
218-
${{ github.workspace }}/test_report/*/diff_plots
217+
${{ github.workspace }}/test_report/*

tests/local/utils/attach_all_plots.bash

+4-4
Original file line numberDiff line numberDiff line change
@@ -3,10 +3,10 @@
33
PR=$1
44
CONFIG=$2
55
cwd=`pwd`
6-
diffs=$GITHUB_WORKSPACE/test_report/$CONFIG/diff_plots
6+
diffs=$GITHUB_WORKSPACE/test_report/$CONFIG
77

88
if [[ ! -d $diffs ]]; then
9-
echo "No diff plots to attach!"
9+
echo "No diff plots to attach at ${diffs}"
1010
exit 0
1111
fi
1212

@@ -24,6 +24,6 @@ cd $diffs
2424
for d in `ls -1`
2525
do
2626
if [[ -d $d && `ls -1 $d` ]]; then
27-
python $cwd/attach_plots_to_pr.py -r $REPO -p $PR -d -t "$TOKEN" --title "$title" $d/*
27+
python $cwd/attach_plots_to_pr.py -r $REPO -p $PR -d -t "$TOKEN" --title "$title" $d/*
2828
fi
29-
done
29+
done

0 commit comments

Comments
 (0)