Reduce stack utilization of TestTemplateTestDescriptor::execute by amaembo · Pull Request #4024 · junit-team/junit-framework (original) (raw)

Not sure about formatting. I've formatted the changed fragment using spotlessApply, even though it's inside the formatter:off section. Probably it worth removing formatter:off.

I think so too. From what I've observed of Spotless over the years, users like JUnit 5 use formatter:(off|on) because the bundled Eclipse formatter can't handle fluent chains like streams very well, so the comments are used as an escape hatch. Thus they should be safe to remove, if the team is happy of course. :)