Skip to content

docs: documentar validacao com testcontainers e politica de retencao no readme - #13

Merged
elielsousa-pathbit merged 1 commit into
masterfrom
docs/testcontainers-and-package-retention
Sep 18, 2026
Merged

elielsousa-pathbit merged 1 commit into
masterfrom
docs/testcontainers-and-package-retention

Conversation

@elielsousa-pathbit

Copy link
Copy Markdown
Contributor

Atualiza o README.md com a documentacao de execucao dos testes com Testcontainers (Option 3) e as informacoes sobre a publicacao automatica no GitHub Packages e politica de retencao das 3 ultimas versoes.

@elielsousa-pathbit
elielsousa-pathbit merged commit a47d0a5 into master Sep 18, 2026
6 checks passed
@elielsousa-pathbit
elielsousa-pathbit deleted the docs/testcontainers-and-package-retention branch September 18, 2026 12:44

@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: 8f9822fbac

ℹ️ 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".

Comment thread README.md


Official multi-architecture Docker images (`linux/amd64` and `linux/arm64`) are published automatically to the GitHub Container Registry (GHCR):
Official multi-architecture Docker images (`linux/amd64` and `linux/arm64`) are published automatically to the GitHub Container Registry (GHCR) on pushes to `master` with changes in `src/`. An automated retention policy maintains strictly the last 3 versions:

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 Correct the stated three-version retention limit

The policy does not maintain strictly three image versions: .github/workflows/release.yml configures keep-n-tagged: 3 while excluding latest, and the workflow comments explicitly state that latest is outside the count. After several releases, the current latest image plus three additional tagged versions can therefore survive, so this new guarantee understates actual registry retention.

Useful? React with 👍 / 👎.

Comment thread README.md
Comment on lines +348 to +349
# Run Testcontainers validation suite
python3 -m unittest tests/test_container.py -v

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 Ensure the documented validation rebuilds the image

When 9rtksync:test already exists locally, this command does not validate the generated image for the current checkout: obter_ou_construir_imagem() in tests/test_container.py returns immediately for any matching cached image and never rebuilds it. A developer rerunning the documented suite after changing src/ or the Dockerfile can therefore get a passing result against stale code; the instructions should explicitly rebuild/remove the tag, or the test should always build the current checkout.

Useful? React with 👍 / 👎.

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