Skip to content

fix: drop v prefix from AWF container image tags#32

Merged
jamesadevine merged 1 commit intomainfrom
devinejames/drop-docker-v
Mar 16, 2026
Merged

fix: drop v prefix from AWF container image tags#32
jamesadevine merged 1 commit intomainfrom
devinejames/drop-docker-v

Conversation

@jamesadevine
Copy link
Collaborator

Uses bare semver tags for Docker images (OCI convention) instead of \�-prefixed tags.

  • \ghcr.io/.../squid:v0.5.0\ → \ghcr.io/.../squid:0.5.0\
  • Tags pulled images as :latest\ for backward compatibility with AWF
  • Display names retain \�\ prefix (human-readable context)

Pull versioned Docker images using bare semver tags (OCI convention)
instead of :latest to ensure reproducible builds, then tag them as
:latest for backward compatibility with AWF.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@jamesadevine jamesadevine merged commit 9c3b85c into main Mar 16, 2026
2 checks passed
@github-actions
Copy link
Contributor

🔍 Rust PR Review

Summary: Looks good — minimal, correct fix already merged.

Findings

✅ What Looks Good

  • The change is applied consistently in both locations in templates/base.yml (the PerformAgenticTask and AnalyzeSafeOutputs/ProcessSafeOutputs jobs) — no missed occurrences.
  • displayName strings intentionally retain the v prefix for human readability while the image tags use bare semver — this distinction is deliberate and correct.
  • No Rust code was touched; the template change is purely cosmetic at the pipeline-generation layer with no risk of introducing unwrap/error-handling regressions.
  • Version bump is consistently applied in both Cargo.toml and Cargo.lock.

No bugs, security concerns, or actionable suggestions — this is a clean, scoped fix.

Generated by Rust PR Reviewer for issue #32 ·

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