Skip to content

chore: Bump symfony/console - #62962

Merged
CarlSchwan merged 1 commit into
masterfrom
carl/symfony/console
Aug 6, 2026
Merged

chore: Bump symfony/console#62962
CarlSchwan merged 1 commit into
masterfrom
carl/symfony/console

Conversation

@CarlSchwan

@CarlSchwan CarlSchwan commented Aug 6, 2026

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

nextcloud/3rdparty#2550

Note that this will breaks some apps as execute now need to return an int, but:

  • This is a private interface in OC\Core
  • Fixing it in an app doesn't break support for older version of NC
  • Most apps already have the correct signature

Alternatively we could patch Command::execute to not return an int like before

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 6, 2026
@CarlSchwan CarlSchwan self-assigned this Aug 6, 2026
@CarlSchwan
CarlSchwan requested a review from a team as a code owner August 6, 2026 10:40
@CarlSchwan CarlSchwan added the 3. to review Waiting for reviews label Aug 6, 2026
@CarlSchwan
CarlSchwan requested review from Altahrim, come-nc, leftybournes, nickvergessen and salmart-dev and removed request for a team August 6, 2026 10:40
Signed-off-by: Carl Schwan <carl@carlschwan.eu>
@CarlSchwan
CarlSchwan force-pushed the carl/symfony/console branch from ea98a29 to e26ef4b Compare August 6, 2026 13:14
@CarlSchwan
CarlSchwan enabled auto-merge August 6, 2026 13:17
@CarlSchwan
CarlSchwan merged commit c75c612 into master Aug 6, 2026
214 of 216 checks passed
@CarlSchwan
CarlSchwan deleted the carl/symfony/console branch August 6, 2026 14:28
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 dependencies technical debt 🧱 🤔🚀

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants