-
-
Notifications
You must be signed in to change notification settings - Fork 4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: Bump symfony/* to 6.4 #48539
build: Bump symfony/* to 6.4 #48539
Conversation
9b38338
to
4e53ecc
Compare
4e53ecc
to
998d972
Compare
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
…dparty bump) Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
Signed-off-by: Joas Schilling <coding@schilljs.com>
…meterOption calls Signed-off-by: Joas Schilling <coding@schilljs.com>
998d972
to
315e465
Compare
@nickvergessen To be precise, I think we can with https://psalm.dev/docs/annotating_code/type_syntax/conditional_types/ , but it’s a bit complicated. Maybe we should simply add in Base: function getQuestionHelper(): QuestionHelper {
/** @var QuestionHelper */
return $this->getHelper('question');
} Seeing how often this is used. |
I'm up for that, if we add a public interface. I don't feel like typing 3rdparty with implied assumptions |
Follow up for nextcloud/3rdparty#1956 and nextcloud/server#48539 Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
/backport e6cd105 to stable29 |
/backport e6cd105 to stable28 |
Summary
Checklist