Skip to content

ci: run the verification block on push and PR #5

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
T-Crypt merged 2 commits into from
Oct 5, 2026
Merged

ci: run the verification block on push and PR #5

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
0949941
ci: run the verification block on push and PR
T-Crypt Oct 5, 2026
2535cbb
fix: Windows host backend, two ctypes bugs the CI job exposed
T-Crypt Oct 5, 2026
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
37 changes: 37 additions & 0 deletions .github/workflows/verify.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,37 @@
name: verify

on:
push:
branches: [main]
pull_request:

jobs:
linux:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: "3.12"
- uses: actions/setup-node@v4
with:
node-version: "20"
- name: collector parses
run: python3 -m py_compile collector/*.py
- name: collector suite
run: python3 -m unittest discover -s collector -p "test_*.py"
- name: ui primitives and ripple state machine
run: node smoke/ui-primitives.mjs
- name: app shell and view model
run: node smoke/shell.mjs

windows:
runs-on: windows-latest
steps:
- uses: actions/checkout@v4
- uses: actions/setup-python@v5
with:
python-version: "3.12"
- name: collector suite (Win32 ctypes backend runs here)
shell: bash
run: python3 -m unittest discover -s collector -p "test_*.py"
6 changes: 3 additions & 3 deletions collector/hostinfo.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -180,7 +180,7 @@ def _win_cpu_times():
if _kernel32.GetSystemTimes(ctypes.byref(idle),
ctypes.byref(kernel),
ctypes.byref(user)):
total = (user.quad + kernel.quad, idle.quad)
total = (user.value + kernel.value, idle.value)
return total, _win_cpu_cores()

def _win_cpu_cores():
Expand Down Expand Up @@ -233,8 +233,8 @@ def _win_processes():
found = []
pids_buf = ctypes.create_string_buffer(65536) # 16384 PIDs
needed = wintypes.DWORD(0)
if not _kernel32.EnumProcesses(pids_buf, ctypes.sizeof(pids_buf),
ctypes.byref(needed)):
if not _psapi.EnumProcesses(pids_buf, ctypes.sizeof(pids_buf),
ctypes.byref(needed)):
return found
n = needed.value // 4
pids = struct.unpack_from("%dI" % n, pids_buf.raw)
Expand Down
Loading