- Issue created by @pooja_sharma
- Merge request !8560issue/3457651: Remove todo views should expect and store a leading. β (Open) created by pooja_sharma
Removed the @todo Views should expect and store a leading /. comments
PLease review, moving to NR
- Status changed to Needs review
2 months ago 4:05pm 27 June 2024 - Status changed to RTBC
2 months ago 11:55pm 27 June 2024 - πΊπΈUnited States smustgrave
Seems like a valid cleanup.
Even though the title says the task think it would be good practice to still include a proposed solution so added that.
- π³πΏNew Zealand quietone New Zealand
Yes, making the complete history available for the review is very helpful. And we need to be sure that the problem the closed issue was to fix has indeed been fixed.
This is ready.
- π³π±Netherlands Lendude Amsterdam
Hmmmm I'm not convinced the referenced issue could be closed....I'll comment there
- π³π±Netherlands Lendude Amsterdam
Reopened the referenced issue, lets see what others think about keeping that open or if others think it can indeed be closed
- Status changed to Postponed
2 months ago 10:00am 5 July 2024