Skip to content

run CI tests on every PR - #625

Merged
RyanPCo merged 2 commits into
mainfrom
ryanco/phase1-ci
Sep 20, 2026
Merged

RyanPCo merged 2 commits into
mainfrom
ryanco/phase1-ci

Conversation

@RyanPCo

@RyanPCo RyanPCo commented Sep 12, 2026 •

Copy link
Copy Markdown
Contributor

.github/workflows/ci.yml, three jobs:

  • lint: ruff check egomimic tests and ruff format --check on the whole
    repo (excluding external/), both with the pre-commit pinned ruff (0.8.6).
  • unit-tests: uv sync --locked from uv.lock (fails if the lock is stale),
    then pytest tests/unit, which includes the Hydra compose+resolve test over
    every shipped config and a real hydra -m launch.
  • wheel: uv build --wheel, install into a fresh venv with --no-deps, import
    from outside the checkout (contents are asserted by test_packaging.py).
  • pyproject pins ruff==0.8.6 so the venv, pre-commit and CI run the same ruff
    (the venv had 0.15, which flags things 0.8.6 does not); uv.lock updated.

Baseline lint cleanup so the gate is green from day one (41 errors, ruff
--fix for 32: unsorted imports, unused imports; by hand: delete two unused
locals, split one semicolon statement, # noqa: E402 on the imports kept
mid-file with the block moved from egomimicUtils.py). No import removed had
side effects. pre-commit's ruff-format then reformatted the touched files, so
five of them (dataset_view.py, stage_processed_folder.py,
aria_to_mano_convert.py, human.py, collect_demo.py) carry whitespace-only
formatting changes beyond the lint fixes.

Whole-repo format: ruff format over every file outside external/ (32 files,
11 of them notebooks, mostly JSON re-indentation), so the format check covers
the whole tree rather than only changed files.

PR auto-review is disabled: its ANTHROPIC_API_KEY account is out of credits and
every run failed. The pull_request trigger is commented out (workflow_dispatch
keeps the file valid); uncomment it once credits are restored. The wiki tracker
is untouched.

Co-Authored-By: Claude Fable 5.1 noreply@anthropic.com
Claude-Session: https://claude.ai/code/session_01UHDiN8yfERcsiFdz4mXuZi

🤖 Generated with Claude Code

https://claude.ai/code/session_01UHDiN8yfERcsiFdz4mXuZi

@RyanPCo
RyanPCo marked this pull request as ready for review September 12, 2026 17:30
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from b738811 to f83c6f6 Compare September 12, 2026 19:39
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-ci branch 2 times, most recently from 3c6f23e to 553a840 Compare September 12, 2026 21:28
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from f83c6f6 to 4944182 Compare September 12, 2026 21:28
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from 4944182 to 4da1f80 Compare September 12, 2026 21:32
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from 4da1f80 to a83d727 Compare September 12, 2026 22:01
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from a83d727 to 84a7fe5 Compare September 12, 2026 22:21
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from 84a7fe5 to d39ce7d Compare September 12, 2026 22:22
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-ci branch 2 times, most recently from 13a2eee to ce80d63 Compare September 12, 2026 22:23
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from d39ce7d to 0baca7b Compare September 12, 2026 22:23
@RyanPCo
RyanPCo force-pushed the ryanco/phase1-env-vars branch from 0baca7b to fd6df11 Compare September 13, 2026 17:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant