Skip to content
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

fix: missing line in _createExportedInjector method #959

Merged

Conversation

eduardoflorence
Copy link

Description

Para fazer o PR #911, eu fiz o fork do repositório do modular e alterei somente o arquivo tracker.dart do modular_core, que já foi mergeado, mas como o modular_core é um package separado, ainda é necessário publicá-lo em pub.dev para que o flutter_modular possa utilizá-lo.
Ao fazer testes após essa alteração, percebi que um novo problema acontece, mas que não tem nenhuma relação com minha alteração.
Descobri que existia uma diferença entre código do modular_core que está na pub.dev (antes da minha alteração) em relação ao que está no repositório do modular atualmente (até a data que foi feito o último upload para a pub.dev).
Uma linha apenas no método _createExportedInjector estava ausente e provocava um problema ao fazer um import de alguma dependência singleton/lazySingleton exportada, pois sempre criava uma nova instância.
Com este PR isto é resolvido e o modular_core já pode ser atualizado na pub.dev

Breaking Change

  • Yes, this is a breaking change.
  • No, this is not a breaking change.

@eduardoflorence eduardoflorence changed the title bug: missing line in _createExportedInjector fix: missing line in _createExportedInjector May 21, 2024
@eduardoflorence eduardoflorence changed the title fix: missing line in _createExportedInjector fix: missing line in _createExportedInjector method May 21, 2024
@jacobaraujo7 jacobaraujo7 merged commit 0f7f8ad into Flutterando:master May 23, 2024
3 of 5 checks passed
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