Closed ostrolucky closed 3 years ago
Looks like the database is being connected to in Laminas\Diagnostics\Check\DoctrineMigration->__construct()
. @cgrabenstein can you confirm? Possible to move the constructor code to the check method?
@kbond yes, seems like this is an oversight on my part. I will provide a fix that moves the constructor code into the check
method.
Awesome, thanks @cgrabenstein!
PR created here: https://github.com/laminas/laminas-diagnostics/pull/20
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
I guess issue here is not needed if referenced PR will solve it
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.
This issue has been automatically closed. Feel free to re-open if it is still relevant.
FYI, the upstream fix has been merged.
Following config works fine (not triggering database)
Following doesn't (tries to connect to database)