Skip to content

refactor: Use new IQueryBuilder::MAX_IN_PARAMETERS - #60960

Merged
Altahrim merged 1 commit into
masterfrom
carl/querybuilder-magic-number
Jun 10, 2026
Merged

refactor: Use new IQueryBuilder::MAX_IN_PARAMETERS#60960
Altahrim merged 1 commit into
masterfrom
carl/querybuilder-magic-number

Conversation

@CarlSchwan

@CarlSchwan CarlSchwan commented Jun 3, 2026

Copy link
Copy Markdown
Member

Summary

TODO

  • ...

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@CarlSchwan
refactor: Use new IQueryBuilder::MAX_IN_PARAMETERS
994565f 
Signed-off-by: Carl Schwan <carlschwan@kde.org>
@CarlSchwan CarlSchwan added this to the Nextcloud 35 milestone Jun 3, 2026
@CarlSchwan CarlSchwan self-assigned this Jun 3, 2026
@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Jun 3, 2026
@CarlSchwan
CarlSchwan requested a review from a team as a code owner June 3, 2026 12:23
@CarlSchwan CarlSchwan added the technical debt 🧱 🤔🚀 label Jun 3, 2026
@CarlSchwan
CarlSchwan requested review from Altahrim, ArtificialOwl, icewind1991 and salmart-dev and removed request for a team June 3, 2026 12:23
@nickvergessen

nickvergessen commented Jun 9, 2026

Copy link
Copy Markdown
Member

It's the first time I see oracle failing with a risky test. Is it related?

@CarlSchwan

CarlSchwan commented Jun 10, 2026

Copy link
Copy Markdown
Member Author

It's the first time I see oracle failing with a risky test. Is it related?

Not related, just something that was failing on master when I opened the PR. Fixed meanwhile

@Altahrim
Altahrim merged commit 2b7415e into master Jun 10, 2026
187 of 197 checks passed
@Altahrim
Altahrim deleted the carl/querybuilder-magic-number branch June 10, 2026 21:09
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 11, 2026
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@nickvergessen nickvergessen nickvergessen approved these changes
@blizzz blizzz blizzz approved these changes
@miaulalala miaulalala miaulalala approved these changes
@hamza221 hamza221 Awaiting requested review from hamza221 hamza221 is a code owner
@SebastianKrupinski SebastianKrupinski Awaiting requested review from SebastianKrupinski SebastianKrupinski is a code owner
@ChristophWurst ChristophWurst Awaiting requested review from ChristophWurst ChristophWurst is a code owner
@tcitworld tcitworld Awaiting requested review from tcitworld tcitworld is a code owner
@ArtificialOwl ArtificialOwl Awaiting requested review from ArtificialOwl ArtificialOwl is a code owner automatically assigned from nextcloud/server-backend
@icewind1991 icewind1991 Awaiting requested review from icewind1991 icewind1991 is a code owner automatically assigned from nextcloud/server-backend
@Altahrim Altahrim Awaiting requested review from Altahrim Altahrim is a code owner automatically assigned from nextcloud/server-backend
@salmart-dev salmart-dev Awaiting requested review from salmart-dev salmart-dev is a code owner automatically assigned from nextcloud/server-backend

Assignees

@CarlSchwan CarlSchwan

Labels

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

Projects

None yet

Milestone

Nextcloud 35

Development

Successfully merging this pull request may close these issues.

5 participants

@CarlSchwan @nickvergessen @blizzz @miaulalala @Altahrim