Skip to content

remove ReconnectWrapper - #25549

Merged
ChristophWurst merged 1 commit into
masterfrom
remove-reconnect-wrapper
Feb 9, 2021
Merged

remove ReconnectWrapper#25549
ChristophWurst merged 1 commit into
masterfrom
remove-reconnect-wrapper

Conversation

@icewind1991

Copy link
Copy Markdown
Member

@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Feb 9, 2021
@icewind1991 icewind1991 added this to the Nextcloud 22 milestone Feb 9, 2021

@ChristophWurst ChristophWurst 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.

🚀

dbal now handles it's own reconnections: https://github.com/doctrine/dbal/blob/3.0.x/UPGRADE.md#the-pingableconnection-interface-is-removed

Signed-off-by: Robin Appelman <robin@icewind.nl>
@icewind1991
icewind1991 force-pushed the remove-reconnect-wrapper branch from 078d170 to 111fa47 Compare February 9, 2021 15:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants