Skip to content

Conversation

Ascenio
Copy link
Contributor

@Ascenio Ascenio commented Apr 20, 2020

Uma pequena mudança que talvez seja breaking-change mas que melhore a legibilidade do Modular.

Considerando que existem o MainModule e o ChildModule, acredito que o ModuleWidget deveria ser renomeado para WidgetModule.

Acho que é uma mudança necessária porque muitas vezes quando fiz uso desse recurso da biblioteca, me encontrei tendo que reescrever o nome do Module duas vezes, já que a primeira coisa que vinha à mente nunca era o nome correto

Ascenio added 2 commits April 20, 2020 11:45
Substituindo ModuleWidget por WidgetModule
@jacobaraujo7
Copy link
Contributor

Muito obrigado pelo pr
@Bwolfs2 o que acha disso?

@Bwolfs2
Copy link
Contributor

Bwolfs2 commented Apr 21, 2020

Soh em uma proxima versao grande com break changes, ou fazer uma classe Casca com o mesmo nome do ModuleWidget pra evitar de parar os codigos que ja estao funcionando.

@AlbertoMonteiro
Copy link
Contributor

Confesso que em relação ao custo beneficio do Breaking Change, não vale a pena mudar esse nome.

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.

4 participants