Skip to content

Commit 811c499

Browse files
authored
Merge pull request #165 from DavidIkov/origin_master
Removed PVS studio from knp: #164
2 parents 45c3f11 + e4eb3c8 commit 811c499

1 file changed

Lines changed: 3 additions & 7 deletions

File tree

.github/workflows/cmake-build.yml

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,13 @@ on:
2121
- '!/docker/**'
2222
- '!/tools/**'
2323
- '!/ci/**'
24-
pull_request_target:
25-
types: [opened, synchronize, reopened]
24+
pull_request:
25+
branches: [ "master" ]
2626

2727
jobs:
2828
linux_build:
2929
runs-on: ${{ matrix.os }}
30-
environment: pr-review-required
30+
3131
strategy:
3232
# Set fail-fast to false to ensure that feedback is delivered for all matrix combinations. Consider changing this to true when your workflow is stable.
3333
fail-fast: false
@@ -86,10 +86,6 @@ jobs:
8686
run: docker run --platform=linux/${{ matrix.arch }} --rm -v ${{ github.workspace }}:/KNP -w /KNP kasperskydh/knp-build-image cmake --build ${{ steps.strings.outputs.build-output-dir }} --parallel ${{ matrix.build_threads_count }}
8787
timeout-minutes: 180
8888

89-
# - name: Analyze C++ code with PVS Studio
90-
# if: matrix.arch == 'amd64'
91-
# run: docker run --platform=linux/${{ matrix.arch }} --rm -v ${{ github.workspace }}:/KNP -w /KNP kasperskydh/knp-build-image bash -c 'pvs-studio-analyzer credentials ${{ secrets.PVS_STUDIO_CREDENTIALS }} && cmake --build ${{ steps.strings.outputs.build-output-dir }} --parallel ${{ matrix.build_threads_count }} --target pvs-analyze'
92-
9389
- name: C++ tests
9490
run: docker run --platform=linux/${{ matrix.arch }} --rm -v ${{ github.workspace }}:/KNP -w /KNP/${{ steps.strings.outputs.build-output-dir }}/knp/tests kasperskydh/knp-build-image ctest -V
9591
timeout-minutes: 180

0 commit comments

Comments
 (0)