ci: Restore pull_request trigger for codeql-analysis.yml
This commit is contained in:
parent
636e3eab96
commit
572cc80577
2 changed files with 12 additions and 5 deletions
4
.github/workflows/ci.yml
vendored
4
.github/workflows/ci.yml
vendored
|
|
@ -105,10 +105,6 @@ jobs:
|
||||||
if: ${{ github.event_name == 'pull_request' }}
|
if: ${{ github.event_name == 'pull_request' }}
|
||||||
uses: ./.github/workflows/valgrind-analysis.yml
|
uses: ./.github/workflows/valgrind-analysis.yml
|
||||||
|
|
||||||
analyse-codeql:
|
|
||||||
needs: [lint-cmake, lint-clang]
|
|
||||||
uses: ./.github/workflows/codeql-analysis.yml
|
|
||||||
|
|
||||||
analyse-sonarcloud:
|
analyse-sonarcloud:
|
||||||
needs: [lint-cmake, lint-clang, pr-comment-flags]
|
needs: [lint-cmake, lint-clang, pr-comment-flags]
|
||||||
if: ${{ needs.pr-comment-flags.outputs.no-sonar != 'true' }}
|
if: ${{ needs.pr-comment-flags.outputs.no-sonar != 'true' }}
|
||||||
|
|
|
||||||
13
.github/workflows/codeql-analysis.yml
vendored
13
.github/workflows/codeql-analysis.yml
vendored
|
|
@ -1,8 +1,19 @@
|
||||||
name: "CodeQL Analysis"
|
name: "CodeQL Analysis"
|
||||||
|
|
||||||
|
# According to the docs, the CodeQL workflow should be triggered directly by push to master
|
||||||
|
# and by pull requests (we only run this on open PRs as it's very slow). We also use the
|
||||||
|
# `workflow_dispatch` event is also enabled to allow manual triggering of the workflow for testing.
|
||||||
|
#
|
||||||
|
# We should not trigger this workflow with `workflow_call` as this causes the error:
|
||||||
|
# "1 configuration present on `master` was not found"
|
||||||
|
#
|
||||||
|
# Sadly, this means we can't roll it into our monolithic CI workflow.
|
||||||
on:
|
on:
|
||||||
workflow_dispatch:
|
workflow_dispatch:
|
||||||
workflow_call:
|
pull_request:
|
||||||
|
types: [opened, synchronize, reopened, ready_for_review]
|
||||||
|
push:
|
||||||
|
branches: [master]
|
||||||
|
|
||||||
jobs:
|
jobs:
|
||||||
analyze:
|
analyze:
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue