| 27 Jan 2022 |
 piegames | The error there was that the commit dropped the package without removing it from all-packages.nix (probably thanks to a rebase conflict fuckup). I don't know if this was the actual error or how the eval for the other platforms managed to even get so far | 22:46:54 |
 Andreas Schrägle | hm. My PR removes every mention of neon_0_29 though. | 23:15:33 |
| 28 Jan 2022 |
| piegames | In reply to @andreas.schraegle:helsinki-systems.de
hm. My PR removes every mention of neon_0_29 though. If you're too stuck on the issue, ping @Nixos/Darwin-Maintainers I guess🤷 | 00:06:02 |
| 30 Jan 2022 |
 Winter (she/her) | if I got ofborg-internal-error added to a PR, but then after pushing some more commits, ofborg was fine, can i remove the label? | 03:09:13 |
| Winter (she/her) | or should i keep it there | 03:09:20 |
 Sandro 🐧 | Please keep it. Normal users can ignore it. It only is a sign that @ofborg eval or a push needs to be done. | 03:40:25 |
| Winter (she/her) | In reply to @sandro:supersandro.de
Please keep it. Normal users can ignore it. It only is a sign that @ofborg eval or a push needs to be done. As I said, I pushed new commits and OfBorg eval'd fine. | 04:57:10 |
| Winter (she/her) | Sandro: so is it fine if I remove it in this case? | 06:40:02 |
| Sandro 🐧 | In reply to @winterqt:nixos.dev
Sandro: so is it fine if I remove it in this case? just ignore the label. You don't need to do anything with it. | 06:40:27 |
| Winter (she/her) | In reply to @sandro:supersandro.de
just ignore the label. You don't need to do anything with it. will do, but why? if it's evaling fine now, why keep it? | 15:02:13 |
| Winter (she/her) | just seems useless to me idk | 15:03:02 |
| Sandro 🐧 | Because it is an indicator which PRs failed and only people with access to ofborg can figure out why it failed. | 15:07:31 |
| Sandro 🐧 | Most of the time those errors have nothing to do with the PR. If so you get a red eval result. More often some GitHub API things have problems. | 15:08:23 |
 hexa | red eval means you broke eval, that's most often the commiters fault, not ofborgs 😀 | 15:25:20 |
| Winter (she/her) | In reply to @sandro:supersandro.de
Because it is an indicator which PRs failed and only people with access to ofborg can figure out why it failed. yes but i never got a red eval, just that label added and an eval didn't start until i pushed again and then everything was fine
so am i keeping it there just because the fact that it failed at one point could be useful to see | 15:41:25 |
| Winter (she/her) | i've seen people remove it when they get it so 🤷♀️ | 15:42:09 |
| Winter (she/her) | actually i guess "people" is cole-h so that makes sense | 15:42:36 |
| hexa | leave it | 15:47:39 |
| hexa | yeah, cole-h is the removal guy 😀 | 15:47:49 |
| hexa | he is important and may do so | 15:47:55 |
 cole-h | In reply to @hexa:lossy.network
he is important and may do so Well... :P | 17:32:34 |
| cole-h | In reply to @winterqt:nixos.dev
yes but i never got a red eval, just that label added and an eval didn't start until i pushed again and then everything was fine
so am i keeping it there just because the fact that it failed at one point could be useful to see FWIW, usually the internal error label is because GitHub sends an API response that, for one reason or another, is missing required fields or is otherwise incomplete | 17:33:45 |
| piegames | The hubcaps library and the GitHub API are slowly diverging | 17:34:18 |
| Winter (she/her) | In reply to @cole-h:matrix.org
FWIW, usually the internal error label is because GitHub sends an API response that, for one reason or another, is missing required fields or is otherwise incomplete figured | 17:35:28 |
| cole-h | In reply to @piegames:matrix.org
The hubcaps library and the GitHub API are slowly diverging I don't doubt that, but it happens not-too-frequently for the volume of PRs we get. Just seems like sometimes their API can't keep up or something... | 17:35:59 |
| cole-h | Given a wealth of free time and, well, wealth, I'd do more investigation... But it hasn't been more harmful than just "you need to restart the eval in one way or another -- no more progress will be made with the current one" | 17:37:04 |
| 2 Feb 2022 |
|  Chinchilla Washington changed their display name from CoilWinder (novus ordo seclorum) to Chuck Winter. | 08:35:51 |
| 13 Feb 2022 |
| hexa | https://github.com/NixOS/nixpkgs/pull/159691 | 21:34:21 |
| hexa | waiting for ofborg for 22h 🙂 | 21:34:25 |
| hexa | probably github webhook shenanigans | 21:35:17 |
