Skip to content

Conversation

@cristian-longhi
Copy link
Contributor

Alterações solicitadas no PR #3781

Descrição

Issue Relacionada

Motivação e Contexto

Como Isso Foi Testado?

Capturas de Tela (se apropriado):

Tipos de Mudanças

  • Bug fix (alteração que corrige uma issue e não altera funcionalidades já existentes)
  • Nova feature (alteração que adiciona uma funcionalidade e não altera funcionalidades já existentes)
  • Alteração disruptiva (Breaking change) (Correção ou funcionalidade que causa alteração nas funcionalidades existentes)

Checklist:

  • Eu li o documento de Contribuição (CONTRIBUTING).
  • Meu código segue o estilo de código deste projeto.
  • Minha alteração requer uma alteração na documentação.
  • Eu atualizei a documentação de acordo.
  • Eu adicionei testes para cobrir minhas mudanças.
  • Todos os testes novos e existentes passaram.

@edwardoliveira
Copy link
Contributor

edwardoliveira commented Sep 3, 2025

@cristian-longhi Pergunta sincera: por que você está outro PR pra endereçar os comentários de review do PR anterior? Não é assim que a gente (ou qualquer outro projeto, até onde eu sei) trabalha! Eu deixei passar uma vez pq achei que tinha algum problema.

As mudanças devem ser feitas no branch de feature do PR original e, assim que as mudanças são enviadas pro GH, o PR original é atualizado. Ficar abrindo novos PRs pra realizar as mudanças sugeridas é muito ineficiente e desnecessário. Favor enviar essas mudanças pro PR original. Eu vou fechar esse.

@edwardoliveira
Copy link
Contributor

Favor fazer as modificações no PR #3781

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.

2 participants