-
Notifications
You must be signed in to change notification settings - Fork 1
fix: behaviours refactoring #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| - streamelements | ||
| - streamholics | ||
| - alinktothehub | ||
| - wildoverflow |
There was a problem hiding this comment.
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?
There was a problem hiding this comment.
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
twitch_bot/cogs/twitch.py
Outdated
| self.silent_initial = silent_initial | ||
|
|
||
| async def sh_person(self, author, channel): | ||
| async def sh_person(self, author, channel: Channel): |
There was a problem hiding this comment.
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.
There was a problem hiding this comment.
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.
RodrigoDornelles
left a comment
There was a problem hiding this 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.
|
alias sugiro alterar o nome do issue ou mergear como "behaviors refactoring" pois combina mais ao proposto dessa bifurcação! |
|
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 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
atualiza cumprimentos
|
closes #17 |
O que foi feito