Skip to content

Corrigir tags GHCR com owner em lowercase no release - #8

Merged
Pantani merged 1 commit into
mainfrom
Pantani/cx/corrigir-falha-no-build
Jun 11, 2026
Merged

Pantani merged 1 commit into
mainfrom
Pantani/cx/corrigir-falha-no-build

Conversation

@Pantani

@Pantani Pantani commented Jun 11, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Normaliza GITHUB_REPOSITORY_OWNER para lowercase ao montar as tags do Docker no workflow de release.
  • Atualiza a documentação de distribuição em EN/PT para refletir as URLs corretas do GHCR.
  • Adiciona regressão cobrindo a geração das tags e evitando o uso direto do owner em maiúsculo.

Testing

  • npm ci
  • npm run lint
  • npm run typecheck
  • npm test
  • npm exec -- vitest run tests/distribution-validation.test.ts
  • npm run docs:build

Summary by CodeRabbit

  • Documentation

    • Updated Docker image tag references to use lowercase naming conventions for consistency across distribution guides.
  • Tests

    • Added validation to ensure Docker image tags follow lowercase naming standards.

@coderabbitai

coderabbitai Bot commented Jun 11, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 7cb21b3a-dc3d-4228-ba8d-879c6d6eae44

📥 Commits

Reviewing files that changed from the base of the PR and between e3060b2 and d16b2e8.

📒 Files selected for processing (4)
  • .github/workflows/release.yml
  • docs/distribution.md
  • docs/pt/distribution.md
  • tests/distribution-validation.test.ts
📜 Recent review details
🧰 Additional context used
📓 Path-based instructions (2)
docs/**/*.md

📄 CodeRabbit inference engine (CLAUDE.md)

Documentation must be written in English at the repo root, with localized VitePress pages only under docs/pt/

Files:

  • docs/distribution.md
  • docs/pt/distribution.md
docs/**/*.{md,mdx}

📄 CodeRabbit inference engine (AGENTS.md)

Documentation must be written in English at repo root; localized VitePress pages live only under docs/pt/

Files:

  • docs/distribution.md
  • docs/pt/distribution.md
🔇 Additional comments (4)
.github/workflows/release.yml (1)

146-151: LGTM!

tests/distribution-validation.test.ts (1)

106-112: LGTM!

docs/distribution.md (1)

116-117: LGTM!

docs/pt/distribution.md (1)

116-117: LGTM!


📝 Walkthrough

Walkthrough

GitHub Actions release workflow, tests, and documentation were updated to consistently use lowercase GitHub organization names when constructing Docker image registry references. The workflow now uses a lowercased OWNER_LC variable instead of the raw GITHUB_REPOSITORY_OWNER, with corresponding test assertions and documentation examples updated to reflect this normalization.

Changes

Docker Registry Tagging Normalization

Layer / File(s) Summary
Workflow Docker tag generation with lowercase owner
.github/workflows/release.yml, tests/distribution-validation.test.ts
Release workflow transforms GITHUB_REPOSITORY_OWNER to lowercase OWNER_LC for constructing ghcr.io/<owner>/ableton-mind:<tag> image references. New test assertion verifies the workflow uses ${OWNER_LC} and does not use the raw GITHUB_REPOSITORY_OWNER form.
English and Portuguese documentation
docs/distribution.md, docs/pt/distribution.md
Distribution documentation examples updated to show lowercase ghcr.io/pantani/... image references for both the v0.1.0 versioned tag and the latest tag.

🎯 1 (Trivial) | ⏱️ ~3 minutes

🐰 A lowercase path to the registry so clear,
No uppercase names will appear here!
From workflows to docs, in Portuguese and more,
The tags are now lowercase forevermore. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title accurately describes the main change: fixing GHCR (GitHub Container Registry) Docker image tags to use a lowercase owner in the release workflow, which aligns with all file modifications across workflow, documentation, and tests.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@Pantani
Pantani merged commit edf7fc2 into main Jun 11, 2026
9 checks passed
@Pantani
Pantani deleted the Pantani/cx/corrigir-falha-no-build branch June 11, 2026 01:27
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