Skip to content

Fix rejected CI/release workflows and validate Actions expressions #7

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
SebRoLENS merged 1 commit into from
Oct 5, 2026
Merged

Fix rejected CI/release workflows and validate Actions expressions #7

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
7 changes: 7 additions & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,13 @@ jobs:
- uses: actions/setup-python@v6
with:
python-version: '3.12'
cache: pip
cache-dependency-path: .github/workflows/ci.yml

- name: Validate GitHub Actions workflows
run: |
python -m pip install actionlint-py==1.7.12.25
actionlint -shellcheck= -pyflakes=

- name: Parse PowerShell source
shell: pwsh
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/cross-platform-ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ jobs:
timeout-minutes: 20
env:
QT_QPA_PLATFORM: offscreen
PACE_CONTROLLER_DATA_DIR: ${{ runner.temp }}/pace-controller-data
steps:
- uses: actions/checkout@v6

Expand All @@ -47,6 +46,8 @@ jobs:
run: python -m pip install -e . -r requirements-ci.txt

- name: Compile and test
env:
PACE_CONTROLLER_DATA_DIR: ${{ runner.temp }}/pace-controller-data
working-directory: cross_platform
run: |
python -m compileall -q src
Expand Down
5 changes: 4 additions & 1 deletion .github/workflows/cross-platform-release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ jobs:
timeout-minutes: 30
env:
QT_QPA_PLATFORM: offscreen
PACE_CONTROLLER_DATA_DIR: ${{ runner.temp }}/pace-controller-data
PYTHONPATH: cross_platform/src
outputs:
version: ${{ steps.prepare.outputs.version }}
Expand Down Expand Up @@ -57,6 +56,8 @@ jobs:
echo "Preparing PACE Controller v$VERSION"

- name: Render and validate the versioned interface
env:
PACE_CONTROLLER_DATA_DIR: ${{ runner.temp }}/pace-controller-data
run: |
python -m pace_controller --language en --screenshot cross_platform/docs/pace_controller_gui.png
python -c "from PySide6.QtGui import QImage; i=QImage('cross_platform/docs/pace_controller_gui.png'); assert not i.isNull() and i.width() >= 1000 and i.height() >= 700, (i.width(), i.height())"
Expand All @@ -71,6 +72,8 @@ jobs:
test -s cross_platform/docs/PACE_Controller_Manual.pdf

- name: Validate prepared release
env:
PACE_CONTROLLER_DATA_DIR: ${{ runner.temp }}/pace-controller-data
run: |
VERSION="${{ steps.prepare.outputs.version }}"
python tests/test_project.py
Expand Down