* bug#69466: Wrong colours for QA @ 2024-02-29 10:49 Andreas Enge 2024-04-22 16:27 ` Christopher Baines 0 siblings, 1 reply; 4+ messages in thread From: Andreas Enge @ 2024-02-29 10:49 UTC (permalink / raw) To: 69466 Hello, it looks like the dark green colour is wrongly chosen in QA, for instance here: https://qa.guix.gnu.org/issue/69441 The issue has been reviewed, but "Comparison unavailable Yet to process revision". I think dark green should only appear when the package is reviewed AND builds correctly (so would be light green without reviewing). Andreas ^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#69466: Wrong colours for QA 2024-02-29 10:49 bug#69466: Wrong colours for QA Andreas Enge @ 2024-04-22 16:27 ` Christopher Baines 2024-04-29 13:42 ` Andreas Enge 0 siblings, 1 reply; 4+ messages in thread From: Christopher Baines @ 2024-04-22 16:27 UTC (permalink / raw) To: Andreas Enge; +Cc: 69466 [-- Attachment #1: Type: text/plain, Size: 755 bytes --] Andreas Enge <andreas@enge.fr> writes: > it looks like the dark green colour is wrongly chosen in QA, > for instance here: > https://qa.guix.gnu.org/issue/69441 > The issue has been reviewed, but "Comparison unavailable > Yet to process revision". > > I think dark green should only appear when the package is reviewed > AND builds correctly (so would be light green without reviewing). When implementing this I chose to have the review trump any other status since hopefully any failing builds will have been taken in to account by the reviewer. I think it's useful to highlight when there's a review regardless of the other information that QA has. 1: https://git.savannah.gnu.org/cgit/guix/qa-frontpage.git/tree/guix-qa-frontpage/issue.scm#n163 [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 987 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#69466: Wrong colours for QA 2024-04-22 16:27 ` Christopher Baines @ 2024-04-29 13:42 ` Andreas Enge 2024-04-29 14:29 ` Christopher Baines 0 siblings, 1 reply; 4+ messages in thread From: Andreas Enge @ 2024-04-29 13:42 UTC (permalink / raw) To: Christopher Baines; +Cc: 69466 Am Mon, Apr 22, 2024 at 05:27:58PM +0100 schrieb Christopher Baines: > When implementing this I chose to have the review trump any other status > since hopefully any failing builds will have been taken in to account by > the reviewer. My impression is that reviews happen often before the package has reached the status of being built on QA. Andreas ^ permalink raw reply [flat|nested] 4+ messages in thread
* bug#69466: Wrong colours for QA 2024-04-29 13:42 ` Andreas Enge @ 2024-04-29 14:29 ` Christopher Baines 0 siblings, 0 replies; 4+ messages in thread From: Christopher Baines @ 2024-04-29 14:29 UTC (permalink / raw) To: Andreas Enge; +Cc: 69466 [-- Attachment #1: Type: text/plain, Size: 667 bytes --] Andreas Enge <andreas@enge.fr> writes: > Am Mon, Apr 22, 2024 at 05:27:58PM +0100 schrieb Christopher Baines: >> When implementing this I chose to have the review trump any other status >> since hopefully any failing builds will have been taken in to account by >> the reviewer. > > My impression is that reviews happen often before the package has reached > the status of being built on QA. I think that's fine too, QA is meant to be helpful not limiting. I do still take this in to account when merging, not pushing things to master if I'd like to see more things build, either just for substitute availability, or because I'm on the lookout for build failures. [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 987 bytes --] ^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2024-04-29 14:29 UTC | newest] Thread overview: 4+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-02-29 10:49 bug#69466: Wrong colours for QA Andreas Enge 2024-04-22 16:27 ` Christopher Baines 2024-04-29 13:42 ` Andreas Enge 2024-04-29 14:29 ` Christopher Baines
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/guix.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).