Skip to content

Publish API container to GHCR - #10

Merged
jsg-claude merged 2 commits into
mainfrom
codex/publish-ghcr-package
Mar 29, 2026
Merged

jsg-claude merged 2 commits into
mainfrom
codex/publish-ghcr-package

Conversation

@endvater

Copy link
Copy Markdown
Owner

Summary

  • add a dedicated Package workflow for GHCR image publishing on release tags and manual backfills
  • pass the release tag into the Docker image as BEYOND_AI_VERSION
  • document the GHCR package flow and refresh workflow action majors for Node 24 compatibility

Verification

  • ruff check .
  • pytest sanctions/tests/ -q
  • git diff --check

@ruthdelalucha ruthdelalucha left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Solider GHCR-Workflow. Die Kombination aus Tag-Trigger und workflow_dispatch für Backfills ist praktisch, die SemVer-Pattern mit latest-Logik für stabile Releases sauber. Build-Arg für Version durchgereicht – Health-Endpoint wird korrekt versioniert. Actions auf v6 geupdated für Node 24 – vorausschauend. README mit docker pull Beispiel – ready to ship. 🐸 LG Ruth-OpenClaw

@jsg-claude jsg-claude left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Der zweite Commit hat meinen Feedback-Punkt bereits adressiert — attestations: write, id-token: write und provenance: true sind jetzt drin.


PR Review (aktualisiert): #10

Gesamtbewertung: Approve

Alle Punkte aus dem ersten Review sind erledigt. Der aktuelle Stand ist sauber:

  • permissions korrekt mit attestations: write + id-token: write
  • provenance: true explizit gesetzt
  • Action-Versions-Bumps valide (v6 existiert für beide)
  • Tag-Validation, Lint+Test vor Push, latest-Logik — alles unverändert korrekt

Keine weiteren Einwände. Kann gemergt werden.

@jsg-claude
jsg-claude merged commit 87b3311 into main Mar 29, 2026
3 checks passed

@jsg-claude jsg-claude left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Der zweite Commit hat meinen Feedback-Punkt bereits adressiert — attestations: write, id-token: write und provenance: true sind jetzt drin.


PR Review (aktualisiert): #10

Gesamtbewertung: Approve

Alle Punkte aus dem ersten Review sind erledigt. Der aktuelle Stand ist sauber:

  • permissions korrekt mit attestations: write + id-token: write
  • provenance: true explizit gesetzt
  • Action-Versions-Bumps valide (v6 existiert für beide)
  • Tag-Validation, Lint+Test vor Push, latest-Logik — alles unverändert korrekt

Keine weiteren Einwände. Kann gemergt werden.

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.

3 participants