Skip to content

Conversation

@HeitorTrielli
Copy link
Contributor

Estou abrindo o PR, mas ainda vou implementar mais código. Quando estiver pronto, vou marcar o @arademaker para revisar.

@HeitorTrielli
Copy link
Contributor Author

@arademaker terminei. O único caveat é que fiz uma versão do deleteQ que retorna optional, porque pode ser o caso em que a gente precise aplicar um deleteQ numa lista nula pelo algoritmo do huffmanPQ

@HeitorTrielli HeitorTrielli changed the title adicionando namespace PQ Adicionando 8.3 - Priority Queues Jul 6, 2025
Copy link
Owner

@arademaker arademaker left a comment

Choose a reason for hiding this comment

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

peço aplicar ajustes. Não esqueça de atualizar o README marcando a seção como 'feita'

@HeitorTrielli
Copy link
Contributor Author

@arademaker implementei as alterações pedidas

Copy link
Owner

@arademaker arademaker left a comment

Choose a reason for hiding this comment

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

segue ultimo pedido para aceitar PR

@HeitorTrielli
Copy link
Contributor Author

@arademaker reverti para namespace em vez de section

@arademaker arademaker merged commit 77bdd3a into main Jul 9, 2025
2 checks passed
@arademaker arademaker deleted the heitor/priority_queues branch July 9, 2025 20:31
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.

3 participants