Skip to content

Commit 89eeca6

Browse files
committed
No longer run security checker as part of integration tests
There's no relation between changes in a PR and the presence of a security issue or between the frequency of PRs and the frequency we want to check for security issues. So remove it here. We keep the script in bin/ for now, might come in handy at some point.
1 parent 24779a2 commit 89eeca6

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

.github/workflows/test-integration.yml

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -122,18 +122,6 @@ jobs:
122122
'
123123
env:
124124
SYMFONY_ENV: ci
125-
- name: Run security audit
126-
if: always()
127-
run: |
128-
cd docker && docker-compose exec -T php-fpm.vm.openconext.org bash -c '
129-
echo -e "\nSensioLabs Security Check\n" && \
130-
./bin/securityChecker.sh && \
131-
cd theme && \
132-
echo -e "\nNPM Audit\n" && \
133-
npm run audit --production
134-
'
135-
env:
136-
SYMFONY_ENV: ci
137125
# - name: Run Cypress visual regression tests
138126
# if: ${{ github.event_name == 'schedule' && matrix.php == env.PROD_PHP }}
139127
# run: |

0 commit comments

Comments
 (0)