Uh oh!
There was an error while loading. Please reload this page.
docs(repo): adicionar template padrão de pull request - #239
Conversation
…ode-Community#230) ## Linear Issue: PAV-119 Closes PAV-119 ## Branch flow - [ ] Este PR é uma feature/fix/chore destinada a `develop`. - [x] Este é um PR de release com origem `develop` e destino `master`. - [x] Este PR não pula o fluxo obrigatório entre `develop` e `master`. ## Objetivo Promover para `master` a implementação do lock distribuído do scraper, garantindo no máximo uma execução ativa do pipeline entre diferentes origens de disparo. ## Resumo das Alterações - Implementa lock distribuído no Valkey compartilhado por cron, execução manual administrativa e cache miss de `/scrape`. - Utiliza aquisição atômica com `SET NX PX`, TTL e renovação periódica. - Protege renovação e liberação por ownership/token. - Implementa comportamento fail-closed quando o Valkey não confirma a aquisição. - Cancela a execução de forma segura em caso de perda do lock. - Adiciona estado operacional com `runId`, origem, início e expiração sem expor o token proprietário. - Adiciona contratos HTTP para execução concorrente e indisponibilidade do run lock. - Alinha os defaults de `SCRAPER_RUN_LOCK_TTL` e `SCRAPER_RUN_LOCK_RENEW_INTERVAL` à semântica fail-fast definida anteriormente. - Adiciona cobertura de testes para concorrência, ownership, configuração, renovação, liberação e cenários de falha. - Atualiza documentação e configuração do Docker Compose. ## Arquivos e Módulos Afetados - Scraper Go - Run lock / Valkey - Scheduler e pipeline do scraper - Backend administrativo - Configuração do scraper - Docker Compose - Testes Go e backend - Documentação operacional ## Validação - [x] Implementação revisada contra o escopo da PAV-119. - [x] Ajustes solicitados no code review aplicados. - [x] Semântica fail-fast das configurações do run lock validada. - [x] Nenhuma alteração de frontend incluída no escopo. - [ ] CI final do PR de release validado. ## Observações Este PR promove alterações já integradas e revisadas em `develop`. Não realizar squash ou alterações adicionais diretamente em `master` fora do fluxo de release.
hltav
left a comment
There was a problem hiding this comment.
A proposta do PR está correta e o template automático faz sentido como melhoria de DX, mas vamos aproveitar esta PAV para consolidar um padrão mais completo para o Candidate.
O contribuition.md já define o conteúdo mínimo esperado das Pull Requests, então o template deve funcionar como um guardrail operacional de qualidade, e não apenas como uma reprodução resumida do guia.
Estou disponibilizando uma versão revisada do pull_request_template.md com seções adicionais para fluxo de branch, validações, impacto em banco/migrations, contratos/API, configuração/infraestrutura, segurança, dados pessoais, dependências, evidências, riscos e rollback.
Por favor, substitua o conteúdo atual pelo template revisado e valide se ele continua sendo carregado corretamente pelo GitHub ao abrir uma nova PR.
Depois dessa adequação, o PR fica pronto para nova revisão.
Jovinull
commented
Aug 29, 2026
Template revisado aplicado conforme solicitado no review. O Commit: Também validei o arquivo com |
Card
Objetivo e escopo
O que foi feito
.github/pull_request_template.md.contribuition.mdpara as regras completas.Módulos afetados
.github/pull_request_template.mdValidação
git diff --checkexecutado sem erros de whitespace.npm run validateexecutado no pre-push: 60 arquivos e 570 testes de backend aprovados; lint do frontend concluído sem erros; build do frontend concluído com sucesso.contribuition.md.Riscos e rollback
Evidências
.github/pull_request_template.mdao criar uma nova Pull Request.