Skip to content

refactor: Inline most magic methods from CoreRequestBuilder#949

Merged
CarlSchwan merged 7 commits into
masterfrom
carl/skim-down-corerequestbuilder
Jul 17, 2026
Merged

refactor: Inline most magic methods from CoreRequestBuilder#949
CarlSchwan merged 7 commits into
masterfrom
carl/skim-down-corerequestbuilder

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

This depends on deprecated APIs from Doctrine/DBAL and follow more the pattern from other nextcloud apps

@CarlSchwan CarlSchwan self-assigned this May 13, 2026
@CarlSchwan
CarlSchwan force-pushed the carl/skim-down-corerequestbuilder branch 3 times, most recently from 7f1e83f to 2e091cd Compare May 20, 2026 08:22
This depends on deprecated APIs from Doctrine/DBAL

Signed-off-by: Carl Schwan <carlschwan@kde.org>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
AI-Assisted-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
CarlSchwan force-pushed the carl/skim-down-corerequestbuilder branch 3 times, most recently from 60dc2df to 18df0b5 Compare May 20, 2026 09:41
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
CarlSchwan force-pushed the carl/skim-down-corerequestbuilder branch from 18df0b5 to 9664d91 Compare May 20, 2026 11:21
As newer versions are not supported on php 8.1

Signed-off-by: Carl Schwan <carlschwan@kde.org>
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan
CarlSchwan force-pushed the carl/skim-down-corerequestbuilder branch from f51b0de to dcc4881 Compare June 8, 2026 13:24
@CarlSchwan
CarlSchwan force-pushed the carl/skim-down-corerequestbuilder branch 4 times, most recently from dd1839f to be1ca0e Compare June 8, 2026 15:21
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the carl/skim-down-corerequestbuilder branch from be1ca0e to 3bf1e7a Compare July 17, 2026 11:05
@CarlSchwan
CarlSchwan merged commit 7875102 into master Jul 17, 2026
31 checks passed
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