Skip to content
This repository has been archived by the owner on Mar 14, 2023. It is now read-only.

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

Closed
petrochenkov opened this issue Mar 17, 2019 · 3 comments

Comments

@petrochenkov
Copy link
Contributor

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
Copy link
Contributor Author

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

@kennytm
Copy link
Member

kennytm commented Mar 17, 2019

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

@pietroalbini
Copy link
Member

Duplicate of rust-lang/rust-log-analyzer#19

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants