Consider using mikepenz/action-junit-report to annotate results on PRs

[G-Rath]

It's easy enough to have rspec output results in junit format, which we could combine with mikepenz/action-junit-report to annotate failing specs

(we could also use this with rubocop as that supports outputting junit too)

[eoinkelly]

I'm assuming are the same kind annotations that our eslint setup already supports. I must admit that I personally don't find much value from them but that might just be me 🤷 Happy to go with what the consensus wants here.

[robotdana]

i personally find it easier to read the output in the log as it's the same as what i am used to when running tests locally, i don't mind the addition but i'm unlikely to look at them

[eoinkelly]

I'm removing ready for dev label for now. I don't think we have much consensus that this feature would be particularly useful. More discussion needed.

[G-Rath]

@eoinkelly we marked this as ready for dev because we got consensus - I believe overall it was more that there weren't any strong disagreements or downsides not to rather that more than a minority would find it useful, but it was consensus none the less.

Fwiw I feel like this was at a guild meeting while you were on leave, but maybe you were there..?

I'll do a PR this week and we can start using it on a few projects.

[G-Rath]

I've created ackama/aws_ec2_environment#9 to showcase this in action