ci: only the light checks run by themselves for now; the full checks and builds by hand #105

Merged
HumanBean merged 1 commit from HumanBean/Longshot:ci/light-checks-only into main 2026-11-13 03:10:18 +00:00
Owner

Why

Every pull request and every push to main started the full checks and the builds, and with one runner they queued behind each other for hours. For now only the light checks run by themselves; the full ones are started by hand when wanted.

What changed

  • ci.yml: the Python source checks and the SPIR-V fixtures run only from Run workflow (if: github.event_name == 'workflow_dispatch'). The title check and workflow lint still run on pull requests and pushes.
  • build-linux.yml, build-windows.yml: no pull_request trigger. The Release workflow still calls them for main's nightly and for tags; a branch can be built by hand.
  • vulkan-dlss.yml: only Run workflow (no push or pull_request trigger).
  • Unchanged: pr-description.yml, third-party.yml, release.yml.
  • CONTRIBUTING.md: says CI runs only the light checks for now, so a pull request carries its own tests/run_all.py results.

Flags and settings

None

Measurements

main: 2a2da67f
this PR: ci/light-checks-only @ f747eed8

main @ 2a2da67f this PR @ f747eed8
tests/run_all.py (Linux) 492 passed, 2 failed, 36 skipped 492 passed, 2 failed, 36 skipped

The 2 failures are the known ones (test_bake_portability: no zstandard; test_builder_log_read_only: no libtk). The checks that read the workflow files pass: test_windows_on_linux, test_appimage_package_retry, test_appimage_player_build, test_linux_docker_packages, test_recomplauncher_json.

Testing

Tested: the suite above on Linux, main and this PR in turn.

Not tested: the workflows on the runner after the change (they run once this is merged and the next pull request or push comes).

Needs other testers: None

Risk and rollback

Breakage is found later: main is no longer checked by itself. Revert this commit to turn the checks back on.

Checklist

For everyone

  • One topic.
  • The title is a Conventional Commit (type(scope): description).
  • Rebased on current main.
  • python -I -B tests/run_all.py run and passing (any failure that also fails on main is named above).
  • Tested in the game, or what wasn't is named under Testing.
  • Testing filled in: what was tested, what wasn't, and what needs other testers (with test requests).
  • Measured: main and this PR at named commits, on the same PC, scene and settings, at least 2 interleaved runs each, the median reported.
  • No game files, dumps, build output or generated files.
  • third_party/ untouched (a new folder with a PROVENANCE.md instead).
  • The license is OK: GPL-3.0-or-later, nothing incompatible and nothing decompiled.

If applicable

  • Docs updated.
## Why Every pull request and every push to `main` started the full checks and the builds, and with one runner they queued behind each other for hours. For now only the light checks run by themselves; the full ones are started by hand when wanted. ## What changed - `ci.yml`: the Python source checks and the SPIR-V fixtures run only from *Run workflow* (`if: github.event_name == 'workflow_dispatch'`). The title check and workflow lint still run on pull requests and pushes. - `build-linux.yml`, `build-windows.yml`: no `pull_request` trigger. The Release workflow still calls them for `main`'s nightly and for tags; a branch can be built by hand. - `vulkan-dlss.yml`: only *Run workflow* (no push or pull_request trigger). - Unchanged: `pr-description.yml`, `third-party.yml`, `release.yml`. - `CONTRIBUTING.md`: says CI runs only the light checks for now, so a pull request carries its own `tests/run_all.py` results. ## Flags and settings None ## Measurements main: 2a2da67f this PR: ci/light-checks-only @ f747eed8 | | main @ 2a2da67f | this PR @ f747eed8 | |---|---|---| | `tests/run_all.py` (Linux) | 492 passed, 2 failed, 36 skipped | 492 passed, 2 failed, 36 skipped | The 2 failures are the known ones (`test_bake_portability`: no zstandard; `test_builder_log_read_only`: no libtk). The checks that read the workflow files pass: `test_windows_on_linux`, `test_appimage_package_retry`, `test_appimage_player_build`, `test_linux_docker_packages`, `test_recomplauncher_json`. ## Testing **Tested:** the suite above on Linux, main and this PR in turn. **Not tested:** the workflows on the runner after the change (they run once this is merged and the next pull request or push comes). **Needs other testers:** None ## Risk and rollback Breakage is found later: `main` is no longer checked by itself. Revert this commit to turn the checks back on. ## Checklist ### For everyone - [x] One topic. - [x] The title is a Conventional Commit (`type(scope): description`). - [x] Rebased on current `main`. - [x] `python -I -B tests/run_all.py` run and passing (any failure that also fails on `main` is named above). - [x] Tested in the game, or what wasn't is named under Testing. - [x] Testing filled in: what was tested, what wasn't, and what needs other testers (with test requests). - [x] Measured: main and this PR at named commits, on the same PC, scene and settings, at least 2 interleaved runs each, the median reported. - [x] No game files, dumps, build output or generated files. - [x] `third_party/` untouched (a new folder with a `PROVENANCE.md` instead). - [x] The license is OK: GPL-3.0-or-later, nothing incompatible and nothing decompiled. ### If applicable - [x] Docs updated.
ci: only the light checks run by themselves for now; the full checks, builds and the DLSS link check are started by hand
Some checks failed
CI / Python source checks (ubuntu-24.04, part 1/2) (pull_request) Has been skipped
CI / Python source checks (ubuntu-24.04, part 2/2) (pull_request) Has been skipped
CI / SPIR-V fixtures (global narrow load/store, direct surface atomics, F2I immediate) (pull_request) Has been skipped
Third-party parts / The same as upstream (pull_request) Has been skipped
PR description / Description follows the template (pull_request_target) Successful in 2s
CI / Conventional Commit title (pull_request) Successful in 2s
CI / Workflow lint (actionlint) (pull_request) Failing after 4s
Third-party parts / As their notes say (pull_request) Successful in 5s
f747eed83c
HumanBean deleted branch ci/light-checks-only 2026-11-13 03:10:18 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
MagicBeanDev/Longshot!105
No description provided.