Skip to content

chore: publica a imagem do servidor no ghcr - #33

Merged
thnbi merged 1 commit into
devfrom
chore/imagem-no-ghcr
Oct 2, 2026
Merged

thnbi merged 1 commit into
devfrom
chore/imagem-no-ghcr

Conversation

@thnbi

@thnbi thnbi commented Oct 2, 2026 •

Copy link
Copy Markdown
Contributor

O que mudou

  • O workflow de release publica a imagem do servidor em ghcr.io/markupp-labs/markupp, com o token do próprio Actions, no lugar de riedelgab/ifsces2 no Docker Hub

Por quê

Depois do merge

  • O pacote no GHCR nasce privado. Um admin da organização precisa torná-lo público em Package settings, para o self-host e o cluster baixarem sem credencial

Como testar

  • Conferir no diff que login, imagem e permissões apontam para o GHCR; a publicação real acontece na próxima tag v*, e o pacote aparece em github.com/orgs/markupp-labs/packages

Auto-review (checklist)

  • Descrição clara (o que/por quê/como testar)
  • PR pequeno e focado
  • Casos limite considerados (ex.: vazio, 0, erro)
  • Evidência de teste (manual ou automatizado)

@thnbi thnbi self-assigned this Oct 2, 2026
@thnbi
thnbi requested a review from NicolasArthurDev October 2, 2026 22:45
@thnbi thnbi added the enhancement New feature or request label Oct 2, 2026

@NicolasArthurDev NicolasArthurDev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

PR pequeno e direto, gostei. A nota em "Depois do merge" sobre deixar o pacote público já resolve o ponto do pull na produção.

Detalhe opcional: no dev, o docs/DEPLOY.md ainda cita riedelgab/ifsces2. O #32 e o #28 já reescrevem esse arquivo, então se eles entrarem logo depois, nem precisa mexer aqui. Aprovado.

@thnbi
thnbi merged commit 01b45b5 into dev Oct 2, 2026
2 checks passed
@thnbi
thnbi deleted the chore/imagem-no-ghcr branch October 2, 2026 23:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants