Skip to content

Initial implementation. - #52

Draft
tyrsson wants to merge 3 commits into
php-db:0.4.xfrom
tyrsson:mysqli-connection-refactor
Draft

Initial implementation.#52
tyrsson wants to merge 3 commits into
php-db:0.4.xfrom
tyrsson:mysqli-connection-refactor

Conversation

@tyrsson

Copy link
Copy Markdown
Member

Testing are passing but not sure how happy I am with it.

Signed-off-by: Joey Smith jsmith@webinertia.net

QA
Documentationno
Bugfixno
BC Breakno
New Featureyes
RFCyes
QAno
House Keepingno

Description

Possible proposal for simplifying the complexity around consuming connection parameters.

Testing are passing but not sure how happy I am with it.
Signed-off-by: Joey Smith <jsmith@webinertia.net>
@tyrssontyrsson self-assigned this Jul 25, 2026
@tyrsson
tyrsson marked this pull request as draft July 25, 2026 08:01
@tyrsson
tyrsson requested a review from simon-mundyJuly 25, 2026 08:02
@tyrsson

Copy link
Copy Markdown
MemberAuthor

Related

php-db/phpdb#146

Signed-off-by: Joey Smith <jsmith@webinertia.net>
Signed-off-by: Joey Smith <jsmith@webinertia.net>

@simon-mundysimon-mundy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obviously needs the linting changes ASAP.

Sign up for freeto 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

@tyrsson@simon-mundy