Closed (fixed)
Project:
Drupal core
Version:
main
Component:
render system
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
9 Mar 2026 at 18:13 UTC
Updated:
16 Apr 2026 at 10:25 UTC
Jump to comment: Most recent
Comments
Comment #3
longwaveComment #4
smustgrave commentedNot sure how to best review this but return types are added and baseline/ignored file is smaller. Pipeline is green
Comment #5
godotislateThere are a couple open suggestions on the MR.
Comment #7
sivaji_ganesh_jojodae commentedI've addressed the suggestion on the MR.
Comment #8
longwaveThe issue was only to add return types to solve the deprecation notices, so there is some scope creep here. In general we try to fix only one thing at a time in an issue.
We also don't allow additional docs alongside @inheritdoc, so I removed that change.
Comment #9
smustgrave commentedThink some of the changes need to be reverted. Since I reverted to Reporter to rerun pipelines for others I can’t close threads but I should have put checkmarks they weren’t needed
Comment #10
sivaji_ganesh_jojodae commented@longwave, thanks for the clarification, and apologies for the earlier scope creep. I had initially included argument type hints along with return types.
I appreciate you updating the issue scope to accommodate both. I’ve now rebased the MR and rerun the pipeline; everything looks good from my side and should be ready for review.
Please let me know if anything else needs to be adjusted.
Comment #11
smustgrave commented@sivaji can you please revert your changes from before. Not the rebase the suggestions you applied please
Comment #12
sivaji_ganesh_jojodae commentedI had interpreted the title update in #8 as including argument types as well—apologies for the confusion. I’ve now reverted those changes and kept only the return types.
Comment #13
smustgrave commentedNo worries at all, its a learning process. Thank you for reverting that.
Comment #14
longwaveComment #15
sivaji_ganesh_jojodae commentedI've added #3582567: Add missing parameter type hints to Twig loader implementations to follow up on the type hint.
Comment #16
smustgrave commentedI’m going to request we don’t open very specific follow ups like that. They’ll have to be well scoped and thought out.
Comment #17
dcam commentedI don't have anything to add. I double-checked all of the return types with the Twig interfaces in GitHub, mostly because of the deprecation notices where it wasn't specific about the affected methods or types. They're all accurate. The MR looks good to me.
Comment #19
amateescu commentedCommitted 5f59e9f and pushed to main. Thanks!