rust-lang/highfive

Feature request: hide previous "your PR failed on Travis" comments when posting a new one

petrochenkov opened this issue · 3 comments

petrochenkov commented

These comments clutter the PR discussion thread, especially when people use CI to test their changes and these comments are created in numerous amount.

Unfortunately even deleted comments are counted by GitHub when it determines the thread as "long" and hides a part of it under the "N hidden items Load more..." button.
So deleting comments doesn't have benefits over hiding, and hiding is preferable because there's still a small chance for the old comments to contain something valuable.

petrochenkov commented

Same applies to old "merge conflict" comments from bors.

kennytm commented

Unfortunately hiding is still not exposed as a GitHub API (isaacs/github#1480).