Skip to content

chore(api): doctrine update#9

Merged
robinlehrmann merged 3 commits intodevelopfrom
feature/doctrine-update
May 27, 2025
Merged

chore(api): doctrine update#9
robinlehrmann merged 3 commits intodevelopfrom
feature/doctrine-update

Conversation

@robinlehrmann
Copy link
Contributor

@robinlehrmann robinlehrmann commented May 27, 2025

The current implementation of src/Monitoring/Sensor/Database/DoctrineDbal.php is incompatible with doctrine/dbal v4. This was fixed in this MR. Additionally symfony version 6.3 was dropped, because it's no longer maintained.

@robinlehrmann robinlehrmann force-pushed the feature/doctrine-update branch 3 times, most recently from b1c6ad7 to 1537243 Compare May 27, 2025 11:00
@robinlehrmann robinlehrmann requested a review from xarem May 27, 2025 11:41
Copy link
Member

@tuxes3 tuxes3 left a comment

Choose a reason for hiding this comment

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

@robinlehrmann this will give a new version v3.0.0

@robinlehrmann robinlehrmann force-pushed the feature/doctrine-update branch from f85eaa3 to 4cd73a8 Compare May 27, 2025 14:06
Robin Lehrmann added 3 commits May 27, 2025 16:22
Updated doctrine/orm to ^3.0.0
doctrine/doctrine-bundle to ^2.14

Signed-off-by: Robin Lehrmann <robin@whatwedo.ch>
This version is no longer maintained.

Signed-off-by: Robin Lehrmann <robin@whatwedo.ch>
Signed-off-by: Robin Lehrmann <robin@whatwedo.ch>
@robinlehrmann robinlehrmann force-pushed the feature/doctrine-update branch from 4cd73a8 to 0911c8b Compare May 27, 2025 14:22
@robinlehrmann
Copy link
Contributor Author

@tuxes3 FYI: I dropped the compatibility with php version 8.1 due to the dependency update. Please let me know if that is fine for you and I will merge this PR 👍

@robinlehrmann robinlehrmann merged commit 105bf5c into develop May 27, 2025
27 of 28 checks passed
@robinlehrmann robinlehrmann deleted the feature/doctrine-update branch May 27, 2025 14:59
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