If there's anything I need to do to move this along, let me know. If it's just that other things are taking a priority, that's fine!

Is the failed build an environmental issue? Looks like it cancelled after waiting 3 hours. Not sure if it can be retried without another push.

Edit: failed again, so might be due to a change in my PR after all. All builds ok locally so a bit stumped.
Edit 2: it's now working 🤷

@smitpatel thanks for initial review, changes are much simpler now.

@maumar this post here may be useful for understanding how I arrived at these changes: #29422 (comment) - thanks

@maumar any feedback?

@ajcvickers is this likely to get reviewed again anytime soon? After @smitpatel's feedback, the changes are quite small and I'm fairly confident in them.

@stevendarby We will get to it when we can, but since Smit is no longer on the team it is taking some time.

@ajcvickers ah I see, thanks for the update

@maumar sorry to pester, but now the 8.0 previews have begun I was just thinking it'd be great to get it into one; and just aware that another 5 months will be too late. Is there any chance of a review over the next few weeks?

@stevendarby I'm planning to take a look at it this week. Apologies for the long delay.

@stevendarby thank you for the contribution!