Skip to content

ci(B3-Pilot): Reusable-CI aus coworkerz-ci (additiv) - #30

Merged
marcohost33-maker merged 1 commit into
mainfrom
claude/b3-pilot-reusable-ci
Jun 4, 2026
Merged

marcohost33-maker merged 1 commit into
mainfrom
claude/b3-pilot-reusable-ci

Conversation

@marcohost33-maker

Copy link
Copy Markdown
Owner

B3-Pilot Python-Typ (Plan 2026-06-03): ruft ci-python.yml@69d3bae (SHA-Pin) via workflow_call. ADDITIV neben bestehender ci.yml - beide muessen gruen sein; Required-Checks unveraendert. Bei Erfolg: Ersatz+BP-Switch als Folge-Welle.

@marcohost33-maker marcohost33-maker added the agent:claude PR authored by Claude Code label Jun 4, 2026
@marcohost33-maker
marcohost33-maker merged commit fa083f2 into main Jun 4, 2026
11 of 12 checks passed
@marcohost33-maker
marcohost33-maker deleted the claude/b3-pilot-reusable-ci branch June 4, 2026 12:15

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 1cdb83f97d

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".


jobs:
gate:
uses: marcohost33-maker/coworkerz-ci/.github/workflows/ci-python.yml@69d3bae0bd54792f499b01c1b6e1e8f6b452742b

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Add the release tag comment to this pinned workflow

This new reusable workflow reference is SHA-pinned, but it omits the version comment required by AGENTS.md (every action reference is @<full-sha> # vX.Y.Z). In workflow/security reviews, the missing human-readable tag makes it harder to verify what vetted release this cross-repository CI workflow is executing; please add the corresponding # v... comment or pin a tagged release SHA that has one.

Useful? React with 👍 / 👎.

uses: marcohost33-maker/coworkerz-ci/.github/workflows/ci-python.yml@69d3bae0bd54792f499b01c1b6e1e8f6b452742b
with:
python-version: "3.12"
install-command: 'pip install -e ".[dev]" pytest-cov'

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Install the qutip extra before running the suite

With this install command, the pilot workflow runs pytest without the qutip extra that the existing CI installs, so the QuTiP cross-checks guarded by qutip_required in tests/test_anchors.py and tests/test_lindblad.py are silently skipped. That makes this “reusable CI” pilot report green while covering less than the current required CI path, and it weakens the anchor/cross-check signal if this workflow is later used for the planned replacement.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:claude PR authored by Claude Code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant