foo-software / lighthouse-check-action

GitHub Action for running @GoogleChromeLabs Lighthouse audits with all the bells and whistles 🔔 Multiple audits, Slack notifications, and more!

https://github.com/marketplace/actions/lighthouse-check

MIT License

478 stars 24 forks source link

PR comments are combined when there are multiple jobs #47

Closed adamhenson closed 3 years ago

adamhenson commented 3 years ago

Summary

When multiple jobs exist using foo-software/lighthouse-check-action@master with PR commenting enabled and when prCommentSaveOld is not set to true... result comments from both jobs are merged to one. See screenshots below. This is reproduced in #45. In this example comments are merged from the the two workflows found in comment-on-push.yml and foo-api.yml.

stale[bot] commented 3 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

adamhenson commented 3 years ago

This was fixed in #45