Skip to content

Add minimal CI - #2

Merged
jagmarques merged 1 commit into
mainfrom
asqav/add-ci
Jul 9, 2026
Merged

Add minimal CI#2
jagmarques merged 1 commit into
mainfrom
asqav/add-ci

Conversation

@jagmarques

@jagmarques jagmarques commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Adds the first CI workflow. The repo had none.

What it does:

  • Runs the pytest suite (15 tests) on Python 3.11 and 3.12
  • Compiles scripts/ as a fast syntax check
  • A ci-ok gate job that needs the test matrix

No package install step. The plugin ships as hook scripts with no pyproject
and no third-party runtime deps, so CI installs pytest and nothing else. No
ruff step either, since the repo carries no lint config to honor.

Tests run with python -m pytest so the repo root lands on sys.path and the
suite can import scripts.asqav_hook. The bare pytest console script cannot,
and one test fails under it.

Triggers on push and pull_request to main.

Proof of work

Local run before push (Python 3.11.15):

$ python -m pytest tests/ -q
...............
15 passed in 21.18s

Live CI on this PR, run https://github.com/jagmarques/asqav-claude-code/actions/runs/29021952576 (conclusion success):

$ gh pr checks 2
ci-ok        pass  2s
test (3.11)  pass  34s
test (3.12)  pass  9s

Both matrix legs installed pytest 9.1.1 and reported the full suite green:

test (3.11): 15 passed in 0.85s
test (3.12): 15 passed in 0.87s

Diff stat: 1 file changed, 34 insertions.

Runs the pytest suite on Python 3.11 and 3.12 plus a compile check, with a
ci-ok gate job. No package install step since the plugin has no pyproject and
needs no pip packages beyond pytest.

Claude-Session: https://claude.ai/code/session_01K1ic1eRNwFgm5re6gANPMZ
@jagmarques
jagmarques marked this pull request as ready for review July 9, 2026 13:39
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@jagmarques
jagmarques merged commit 11f883d into main Jul 9, 2026
3 checks passed
@jagmarques
jagmarques deleted the asqav/add-ci branch July 9, 2026 13:40
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