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

Update Manager.php #109

Closed
wants to merge 2 commits into from
Closed

Update Manager.php #109

wants to merge 2 commits into from

Conversation

lpj145
Copy link

@lpj145 lpj145 commented Sep 15, 2017

Agora sim, depois que compreendi melhor o código, o erro se encontrava na ordem de inicialização da função init. Pool depende do Container pronto, e o mesmo não estava disponível ao instanciar.

Agora sim, com a melhor compreensão do código, o erro se encontrava na ordem de inicialização da função init. Pool depende do Container pronto e o mesmo não estava disponível ao instanciar.
@coveralls
Copy link

coveralls commented Sep 15, 2017

Coverage Status

Coverage decreased (-9.2%) to 88.279% when pulling 38ae35f on lpj145:patch-2 into 5b9956f on leroy-merlin-br:master.

@leroy-merlin-br leroy-merlin-br deleted a comment Sep 15, 2017
@lpj145
Copy link
Author

lpj145 commented Sep 15, 2017

Nunca usei travis, desculpa pelos logs infinitos, mas agora deu certinho, só alterar a documentação, melhor do que cair o coverage. :)

@lpj145 lpj145 mentioned this pull request Sep 15, 2017
@coveralls
Copy link

coveralls commented Sep 15, 2017

Coverage Status

Coverage decreased (-4.9%) to 92.56% when pulling e80d5a2 on lpj145:patch-2 into 5b9956f on leroy-merlin-br:master.

@lpj145
Copy link
Author

lpj145 commented Sep 15, 2017

Sério, quero entender isso, uma alteração pequenina baixou tanto assim ?

@lpj145 lpj145 closed this Sep 23, 2017
@lpj145 lpj145 deleted the patch-2 branch September 23, 2017 03:21
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.

2 participants