Refactor nemli.commands.summarize:response_to_list function

milharal-dev / nemli-nemlerei-bot

Um bot de discord para criar resumos baseado nas últimas mensagens de um canal

8 stars 1 forks source link

Refactor nemli.commands.summarize:response_to_list function #40

Closed ryukinix closed 1 month ago

ryukinix commented 1 month ago

          Suggestion:

Acho q da para diminuir essa complexidade aqui, pq a única coisa q ta mudando é o summary[:x] e summary[x:]

Criar duas vars com esses caras e ter um único return no fim com toda o resto do code.

_Originally posted by @rdenadai in https://github.com/milharal-dev/nemli-nemlerei-bot/pull/35#discussion_r1691451737_

luksquaresma commented 1 month ago

Adicionando: tamanho limite de mensagem dinâmico para considerar já o header.

BUG: mudanças de header para além do tamanho padrão (100 char) quebram a separação de msgs.

ryukinix commented 1 month ago

Boa ideia

Em qui., 25 de jul. de 2024 23:22, Lucas Quaresma @.***> escreveu:

Adicionando: tamanho limite de mensagem dinâmico para considerar já o header e evitar bugs com mudanças futuras com headers maiores.

— Reply to this email directly, view it on GitHub https://github.com/milharal-dev/nemli-nemlerei-bot/issues/40#issuecomment-2251843237, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2J57XSR4PHAADMSWE4NELZOGXG7AVCNFSM6AAAAABLPNAEIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENJRHA2DGMRTG4 . You are receiving this because you authored the thread.Message ID: @.***>