Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/cmake-linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,12 +32,12 @@
- uses: actions/checkout@v7
with:
fetch-depth: 0
- uses: actions/setup-python@v6
- uses: actions/setup-python@v7
with:
python-version: '3.10.11'

- name: "Install jsondiff"
run: pip install jsondiff

Check warning on line 40 in .github/workflows/cmake-linux.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--only-binary :all:" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=QIICR_dcmqi&issues=AZ-VhRixDGe5_1PMm39s&open=AZ-VhRixDGe5_1PMm39s&pullRequest=553

Check warning on line 40 in .github/workflows/cmake-linux.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=QIICR_dcmqi&issues=AZ-VhRixDGe5_1PMm39t&open=AZ-VhRixDGe5_1PMm39t&pullRequest=553
- name: "Install cmake"
run: |
sudo apt-get update
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/cmake-macos.yml
Original file line number Diff line number Diff line change
Expand Up @@ -40,18 +40,18 @@
- uses: actions/checkout@v7
with:
fetch-depth: 0
- uses: actions/setup-python@v6
- uses: actions/setup-python@v7
with:
python-version: '3.10.11'

# Install build tools (cmake and ninja) as well as jsondiff
- name: "Install jsondiff"
run: pip install jsondiff

Check warning on line 49 in .github/workflows/cmake-macos.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Using dependencies without locking resolved versions is security-sensitive.

See more on https://sonarcloud.io/project/issues?id=QIICR_dcmqi&issues=AZ-VhRlLDGe5_1PMm39v&open=AZ-VhRlLDGe5_1PMm39v&pullRequest=553

Check warning on line 49 in .github/workflows/cmake-macos.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Omitting "--only-binary :all:" can lead to the execution of setup scripts. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=QIICR_dcmqi&issues=AZ-VhRlLDGe5_1PMm39u&open=AZ-VhRlLDGe5_1PMm39u&pullRequest=553
- name: "Install cmake"
run: |
cmake --version
echo ${PATH}
wget --no-check-certificate https://github.com/ninja-build/ninja/releases/download/v1.12.1/ninja-mac.zip

Check warning on line 54 in .github/workflows/cmake-macos.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Not disabling redirects might allow for redirections to insecure websites. Make sure it is safe here.

See more on https://sonarcloud.io/project/issues?id=QIICR_dcmqi&issues=AZ-VhRlLDGe5_1PMm39w&open=AZ-VhRlLDGe5_1PMm39w&pullRequest=553

Check failure on line 54 in .github/workflows/cmake-macos.yml

View check run for this annotation

SonarQubeCloud / SonarCloud Code Analysis

Enable server certificate validation on this SSL/TLS connection.

See more on https://sonarcloud.io/project/issues?id=QIICR_dcmqi&issues=AZ-VhRlLDGe5_1PMm39x&open=AZ-VhRlLDGe5_1PMm39x&pullRequest=553
unzip ninja-mac.zip && sudo cp ninja /usr/local/bin/

# Make sure all directories required for computing the hash actually exist
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/cmake-win.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ jobs:
vs-version: '17'
msbuild-architecture: x64

- uses: actions/setup-python@v6
- uses: actions/setup-python@v7
with:
python-version: '3.10.11'

Expand Down