WordPress / action-wp-playground-pr-preview

MIT License
5 stars 1 forks source link

Mode switcher: New comment vs updating the issue description #14

Open adamziel opened 3 months ago

adamziel commented 3 months ago

It would be useful to select one of a few actions:

@youknowriad provided a good rationale for that in the related Gutenberg PR:

With this, it's going to be the sixth bot comment on PRs on Gutenberg. All of the comments are useful but their number makes it so that it's very noisy and distract a lot from the PR.

We have:

A comment to welcome new contributors A comment to tell new contributors that they're missing the connection in WP profiles (yes, I think there are two comments for new contributors) A comment from the props bot A comment about the bundle size A comment about flaky tests A comment about modified php files A comment about the previews on Playground And we also discussed adding a comment about the "performance job results" (link to summary) I like all of these comments and I actually have ideas to add more comments but this is not sustainable IMO for PR authors and reviewers. We should explore merging all of these comments into:

One pre-merge comment. One post-merge comment. And smartly use things like "details" to hide/show sections.

(I don't consider this comment a blocker for this PR, but I think we should explore that soonish)