Repository navigation
fix(ci): run cleanly on the self-hosted runner - #112
Merged
Merged
Conversation
The SurrealDB server job's container now has a name per run, takes any free port and is removed at the end, so a cancelled job's leftover "surreal" container no longer fails the next one. Jest's per-test timeout is 20 s: three UI tests took over 5 s on the runner. Also sorts an import and formats two Fedora test files main's lint rejects. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JifG4FUAcs7EFGrcotkEDz
mypy on main rejects getaddrinfo's str | int address in a tuple[str, ...]. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JifG4FUAcs7EFGrcotkEDz
…out posix_spawn setsid uv's Python builds (CI's 3.12.5) raise NotImplementedError for setsid=True, so Chromium never started there; fall back to setpgroup=0. The calendar redirect test pins localhost to its IPv4 address, as Ubuntu also gives it ::1. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01JifG4FUAcs7EFGrcotkEDz
Merged
6 of 8 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Before: on the Ubuntu runner every CI run went red. The SurrealDB server job failed at once because a cancelled job's
surrealcontainer was still there. Three Next.js UI tests went past Jest's 5 s timeout. The backend lint failed on an unsorted import that is already on main.After: the SurrealDB container gets a name for each run and binds any free local port. It is removed at the end even when the job fails or is cancelled. Jest allows 20 s per test. The Fedora test files pass
ruff checkandruff format --check.How:
LENS_TEST_SURREAL_URLis now read fromdocker portand written to$GITHUB_ENV. Anif: always()step runsdocker rm -f.testTimeoutis set injest.config.ts.ruff formatwas run on the two Fedora test files.🤖 Generated with Claude Code
https://claude.ai/code/session_01JifG4FUAcs7EFGrcotkEDz
Generated by Claude Code