这是indexloc提供的服务,不要输入任何密码
Skip to content

Conversation

@vcwild
Copy link
Owner

@vcwild vcwild commented Sep 1, 2021

O que foi feito

  • O bot agora não mais procurará por pessoas na sala para interagir
  • As interações do bot agora são reativas (dependem do envio de mensagens)

@vcwild vcwild added the enhancement New feature or request label Sep 1, 2021
@vcwild vcwild requested a review from caio-emidio September 1, 2021 22:39
@vcwild vcwild marked this pull request as draft September 1, 2021 22:40
@vcwild vcwild added the duplicate This issue or pull request already exists label Sep 1, 2021
@vcwild vcwild marked this pull request as ready for review September 1, 2021 22:47
@vcwild vcwild requested review from RodrigoDornelles and removed request for caio-emidio September 6, 2021 16:25
- streamelements
- streamholics
- alinktothehub
- wildoverflow

Choose a reason for hiding this comment

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

tem alguma diferença esta alteração?

Copy link
Owner Author

Choose a reason for hiding this comment

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

Não tem atualização prática, só é um lint básico pra deixar em ordem alfabética

self.silent_initial = silent_initial

async def sh_person(self, author, channel):
async def sh_person(self, author, channel: Channel):

Choose a reason for hiding this comment

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

se for fortificar a tipagem, pode fazer com author também.

Copy link
Owner Author

Choose a reason for hiding this comment

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

Posso fazer isso sim, vai trazer uma maior clareza pro código.

Copy link

@RodrigoDornelles RodrigoDornelles left a comment

Choose a reason for hiding this comment

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

foram feitas boas melhorias técnicas, está mais simples de entender.

@RodrigoDornelles
Copy link

alias sugiro alterar o nome do issue ou mergear como "behaviors refactoring" pois combina mais ao proposto dessa bifurcação!

@vcwild vcwild changed the title fix: alter bot event behaviour fix: behaviours refactoring Sep 9, 2021
@vcwild
Copy link
Owner Author

vcwild commented Sep 9, 2021

usei tipos primitivos como anotadores genéricos de parâmetros onde a tipagem esperada será de fato um tipo primitivo e não uma dataclass (e.g. tipo primitivo author: str ao inves de dataclass author: Author) seguindo ainda as convenções mais atuais do python 3.9 onde generics não utilizam mais a lib de typing

Copy link
Owner Author

@vcwild vcwild left a comment

Choose a reason for hiding this comment

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

atualiza cumprimentos

@vcwild
Copy link
Owner Author

vcwild commented Sep 9, 2021

closes #17

@vcwild vcwild merged commit 449e500 into main Sep 9, 2021
@vcwild vcwild deleted the fix/event-behaviour branch September 9, 2021 22:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

duplicate This issue or pull request already exists enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants