Skip to content

Commit 804f25f

Browse files
author
Gonzalo Diaz
committed
[BUGFIX] sonarcloud: Omitting "--only-binary :all:" can lead to the execution of setup scripts. Make sure it is safe here.
1 parent 054e331 commit 804f25f

5 files changed

Lines changed: 17 additions & 156 deletions

File tree

.github/workflows/python-coverage.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -44,22 +44,22 @@ jobs:
4444
4545
- name: Test an coverage collect
4646
run: >
47-
python3 -m coverage run -m pytest --verbose
47+
pipenv run coverage run -m pytest --verbose
4848
-o log_cli=true
4949
--log-cli-level=INFO
5050
src/
5151
5252
- name: Coverage Report
5353
run: |
54-
python3 -m coverage report
54+
pipenv run coverage report
5555
5656
- name: Coverage lcov (codecov)
5757
run: |
58-
python3 -m coverage lcov -o coverage/lcov.info
58+
pipenv run coverage lcov -o coverage/lcov.info
5959
6060
- name: Coverage xml (sonarcloud)
6161
run: |
62-
python3 -m coverage xml -o coverage/coverage.xml
62+
pipenv run coverage xml -o coverage/coverage.xml
6363
6464
- name: Upload coverage artifact
6565
uses: actions/upload-artifact@v7

.github/workflows/python-lint.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ jobs:
4242
- name: Install tooling on Windows
4343
if: runner.os == 'Windows'
4444
run: |
45-
choco install pipenv -y
45+
python -m pip install --user pipenv==2026.6.1
4646
shell: pwsh
4747

4848
- name: Tooling check
@@ -58,20 +58,20 @@ jobs:
5858
5959
- name: Lint (pylint)
6060
run: |
61-
pylint --verbose --recursive yes src/
61+
pipenv run pylint --verbose --recursive yes src/
6262
6363
- name: Lint (flake8)
6464
run: |
65-
python3 -m flake8 --verbose src/
65+
pipenv run flake8 --verbose src/
6666
6767
- name: Lint (pyright) static type checker
6868
run: |
69-
python3 -m pyright --verbose src/
69+
pipenv run pyright --verbose src/
7070
7171
- name: Styling (pycodestyle)
7272
run: |
73-
python3 -m pycodestyle --statistics src/
73+
pipenv run pycodestyle --statistics src/
7474
7575
- name: Styling (autopep8)
7676
run: |
77-
python3 -m autopep8 --diff --recursive --exit-code --verbose .
77+
pipenv run autopep8 --diff --recursive --exit-code --verbose .

.github/workflows/python-test.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ jobs:
4545
- name: Install tooling on Windows
4646
if: runner.os == 'Windows'
4747
run: |
48-
choco install pipenv -y
48+
python -m pip install --user pipenv=2026.6.1
4949
shell: pwsh
5050

5151
- name: Tooling check
@@ -61,11 +61,11 @@ jobs:
6161
6262
- name: Test
6363
run: >
64-
coverage run -m pytest --verbose
64+
pipenv run coverage run -m pytest --verbose
6565
-o log_cli=true
6666
--log-cli-level=INFO
6767
src/
6868
6969
- name: Coverage
7070
run: |
71-
coverage report
71+
pipenv run coverage report

Dockerfile

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,10 @@ ENV WORKDIR=/app
1616
WORKDIR ${WORKDIR}
1717

1818
COPY ./Makefile ${WORKDIR}/
19-
COPY ./requirements.txt ${WORKDIR}/
19+
COPY Pipfile ${WORKDIR}/
20+
COPY Pipfile.lock ${WORKDIR}/
21+
22+
RUN python -m pip install --user pipenv==2026.6.1
2023
RUN make dependencies
2124

2225
###############################################################################

PIPENV_MIGRATION.md

Lines changed: 0 additions & 142 deletions
This file was deleted.

0 commit comments

Comments
 (0)