-
Notifications
You must be signed in to change notification settings - Fork 75
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
Use DBIx::Connector in the non-PSGI case #577
Labels
Milestone
Comments
Discussed this with @svenoe . There is no good reason to use DBIx::Connector only when running under a webserver. Thus the code can be simplified and DBIx::Connector can be used in all cases. |
bschmalhofer
added a commit
that referenced
this issue
Nov 11, 2021
thus they shouldn't be listed under the section for the feature 'gazelle'.
bschmalhofer
added a commit
that referenced
this issue
Nov 11, 2021
no special case when not running in a web environment
bschmalhofer
added a commit
that referenced
this issue
Nov 11, 2021
bschmalhofer
added a commit
that referenced
this issue
Nov 11, 2021
bschmalhofer
added a commit
that referenced
this issue
Nov 11, 2021
The test suite looked fine. Closing this issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Reducing the number of variants is a good thing.
But it has to be pondered whether DBIx::Connector should really be used for console commands and daemon tasks.
The text was updated successfully, but these errors were encountered: