16 Jun 2021 |
@grahamc:nixos.org | :) | 13:43:54 |
Sandro | I think it is a dupe of https://github.com/NixOS/hydra/issues/917 | 13:47:11 |
@grahamc:nixos.org | a bit of an annoyin bug :) | 13:52:03 |
@grahamc:nixos.org | okay this is going to ruin my day | 14:34:46 |
Sandro | glad you liked my finding 😂 | 14:35:01 |
@grahamc:nixos.org | jobsetevalmembers doesn't have an id field, and I'm guessing the DBIx query generator decided to change what "me" was at some point | 14:35:37 |
@grahamc:nixos.org | ohhh | 14:36:13 |
@grahamc:nixos.org | I see | 14:36:13 |
@grahamc:nixos.org | I broke it | 14:36:14 |
Sandro |  Download image.png | 14:40:28 |
@grahamc:nixos.org | ORMs are the best and worst thing | 15:09:36 |
@grahamc:nixos.org | RFR: https://github.com/NixOS/hydra/pull/973 maybe ajs124 / das_j would like to look-see :) | 15:35:47 |
ajs124 | oh no, why me | 15:39:53 |
@grahamc:nixos.org | you know what you did | 15:42:49 |
das_j | why me? | 15:47:01 |
das_j | Like isn't there a name that speaks more for itself? | 15:47:11 |
ajs124 | it literally took me until now to finally find the link in the UI to this, so I didn't need to construct a url manually | 15:50:11 |
@grahamc:nixos.org | I actually don't know where the link to this page is | 15:51:14 |
das_j | also (I know this isn't related to your change and I'd just be annyoing everyone for requesting you to change it) why are we naming the evals list in line 231 the same as the evals scalar in 227? | 15:51:14 |
@grahamc:nixos.org | great question, let's change that | 15:52:01 |
ajs124 | go to a build, in the summary, there's "Part of:". don't click that link, click the "(and N others)". that's it. | 15:52:08 |
das_j | ajs124: Can you give an example link? | 15:52:29 |
das_j | Or is our Hydra unaffected? | 15:52:38 |
ajs124 | we're affected, but seeing as neither of us knew how to even get there, I think we're fine | 15:54:16 |
das_j | Got it: https://hydra.nixos.org/build/144936736/evals | 15:54:20 |
@grahamc:nixos.org | https://github.com/NixOS/hydra/pull/973/commits/5d95abf54026ad1901da30fde52d614d91b9800f | 15:55:12 |
das_j | grahamc (he/him): The new name is actually more confusing :D But now that I know what the PR actually does, it does look good | 15:55:14 |
@grahamc:nixos.org | do you have a nicer name? | 15:58:12 |
das_j | I don't even know what it's supposed to contain :D | 15:58:26 |
@grahamc:nixos.org | I think one of these: https://metacpan.org/pod/DBIx::Class::ResultSet | 15:58:42 |