Skip to content

Commit 2d85812

Browse files
authored
Only force enabled for vulnerability analysis if model_gs is present
1 parent 200757d commit 2d85812

File tree

1 file changed

+7
-8
lines changed

1 file changed

+7
-8
lines changed

.github/workflows/reverse_engineering.yml

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,6 @@ jobs:
9595
fi
9696
9797
- name: Check for gold standard
98-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
9998
run: |
10099
# only analyze vulnerabilities if a gold standard is present
101100
if [ -d "benchmark/${{ matrix.directory }}/model_gs" ]; then
@@ -105,7 +104,7 @@ jobs:
105104
fi
106105
107106
- name: Checkout repositories
108-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
107+
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || env.analyze_vulnerabilities == 'true' }}
109108
run: |
110109
# Assuming repositories is a comma-separated list
111110
IFS=',' read -r -a repositories_array <<< "${{ env.repositories }}"
@@ -116,7 +115,7 @@ jobs:
116115
done
117116
118117
- name: Run Retriever
119-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
118+
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || env.analyze_vulnerabilities == 'true' }}
120119
uses: PalladioSimulator/Palladio-ReverseEngineering-Retriever@main
121120
with:
122121
source_path: repo/${{ matrix.directory }}
@@ -128,14 +127,14 @@ jobs:
128127
rules_path: ${{ env.rules_path }}
129128

130129
- name: Download results
131-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
130+
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || env.analyze_vulnerabilities == 'true' }}
132131
uses: actions/download-artifact@v3
133132
with:
134133
name: retriever
135134
path: results
136135

137136
- name: Prepare upload
138-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
137+
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || env.analyze_vulnerabilities == 'true' }}
139138
run: |
140139
mkdir -p to_commit/${{ matrix.directory }}/model_re/pcm
141140
mkdir -p to_commit/${{ matrix.directory }}/model_re/uml
@@ -147,7 +146,7 @@ jobs:
147146
mv results/repo/${{ matrix.directory }}/* to_commit/${{ matrix.directory }}/model_re/pcm
148147
149148
- name: Render UML diagrams
150-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
149+
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || env.analyze_vulnerabilities == 'true' }}
151150
run: |
152151
sudo apt-get install graphviz
153152
wget https://github.com/plantuml/plantuml/releases/download/v1.2023.13/plantuml-1.2023.13.jar -O plantuml.jar
@@ -158,15 +157,15 @@ jobs:
158157
done
159158
160159
- name: Update version
161-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
160+
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || env.analyze_vulnerabilities == 'true' }}
162161
run: |
163162
# manually overwrite current_version to avoid bug with yq
164163
sed -i 's/current_version:.*/current_version: "${{ needs.collect_info.outputs.latest_version }}"/' "benchmark/${{ matrix.directory }}/.retriever.yml"
165164
# yq '.current_version = "${{ needs.collect_info.outputs.latest_version }}"' -i "benchmark/${{ matrix.directory }}/.retriever.yml"
166165
mv "benchmark/${{ matrix.directory }}/.retriever.yml" to_commit/${{ matrix.directory }}/.retriever.yml
167166
168167
- name: Upload artifact
169-
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || inputs.analyzeVulnerabilities == 'true' }}
168+
if: ${{ needs.collect_info.outputs.latest_version != env.current_version || inputs.overrideBenchmark == 'true' || env.analyze_vulnerabilities == 'true' }}
170169
uses: actions/upload-artifact@v3
171170
with:
172171
name: to_commit

0 commit comments

Comments
 (0)