Skip to content

Remove most usage of query exception - #62882

Open
CarlSchwan wants to merge 1 commit into
masterfrom
carl/query-exception
Open

Remove most usage of query exception#62882
CarlSchwan wants to merge 1 commit into
masterfrom
carl/query-exception

Conversation

@CarlSchwan

Copy link
Copy Markdown
Member

Summary

Only remaining are the actual throws

TODO

  • ...

Checklist

AI (if applicable)

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

@CarlSchwan CarlSchwan added this to the Nextcloud 35 milestone Aug 4, 2026
@CarlSchwan CarlSchwan self-assigned this Aug 4, 2026
@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Aug 4, 2026
@CarlSchwan
CarlSchwan requested a review from hamza221 as a code owner August 4, 2026 12:53
@CarlSchwan CarlSchwan added the technical debt 🧱 🤔🚀 label Aug 4, 2026
@CarlSchwan
CarlSchwan requested review from Altahrim, nfebe, salmart-dev and sorbaugh and removed request for a team August 4, 2026 12:53
@CarlSchwan
CarlSchwan force-pushed the carl/query-exception branch 2 times, most recently from 38702df to bc26308 Compare August 4, 2026 16:03
@nickvergessen
nickvergessen removed their request for review August 6, 2026 08:52
Last step before moving QueryException in lib/private and making this an
internal class.

Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the carl/query-exception branch from bc26308 to 1363e22 Compare August 10, 2026 13:54
@CarlSchwan
CarlSchwan enabled auto-merge August 10, 2026 14:19
@ChristophWurst ChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Aug 10, 2026
@miaulalala
miaulalala removed their request for review August 11, 2026 09:32
@nextcloud-bot nextcloud-bot mentioned this pull request Aug 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to release Ready to be released and/or waiting for tests to finish technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants