| 7 Feb 2022 |
 @grahamc:nixos.org | When I broke your hydra on purpose did you adapt or partially revert, or? | 18:37:25 |
 das_j | well adapt. there's no point in carrying patches for years and having to rebase them over time | 18:43:24 |
| das_j | also your changes did usually improve things so staying with the old working state was not ideal either | 18:43:42 |
| @grahamc:nixos.org | I just wanted to make sure it wasn’t a nuisance you were actively choosing to avoid. That wouldn’t be very nice! | 18:44:22 |
| das_j | nono, the only time we reverted your changes was when the signature stuff broke our hydra | 18:44:59 |
| das_j | I swear I never disliked your changes :D | 18:45:09 |
| @grahamc:nixos.org | whew, thanks :) | 18:48:08 |
| das_j | Btw, is both dynamic runcommand and runcommand logs on master now? | 18:48:31 |
| das_j | We could upgrade then and drop the patches (and my insecure implementation 👀 | 18:48:47 |
| das_j | * We could upgrade then and drop the patches (and my insecure implementation 👀) | 18:48:48 |
 cole-h | Dynamic RunCommand, not yet. But the logs are on (Hydra) master. | 18:53:03 |
| cole-h | (Dynamic RunCommand PR: https://github.com/NixOS/hydra/pull/1103) | 18:53:16 |
| @grahamc:nixos.org | I'm a bit shy on merging the dynamic runcommand PR given, well, its behavior :P | 18:53:34 |
| @grahamc:nixos.org | we use it internally but I'd like at least another set of people want it for real before merging | 18:54:00 |
| das_j | aren't you using it, ajs124? | 18:54:16 |
 ajs124 | yes | 18:54:43 |
| das_j |
url = "https://nc.helsinki.tools
big oof
| 18:55:04 |
| @grahamc:nixos.org | neat | 18:55:19 |
| @grahamc:nixos.org | is it a good interface? | 18:55:59 |
| ajs124 | I'm not sure I'm using it properly, but I'd say it's not worse than global/static/classic runcommand | 18:57:02 |
| @grahamc:nixos.org | well that's a start | 18:57:16 |
| @grahamc:nixos.org | event cleanup PR: https://github.com/NixOS/hydra/pull/1144 | 23:24:19 |
| @grahamc:nixos.org | mostly docs if anyone wants to give that a review | 23:24:25 |
| @grahamc:nixos.org | adding support for those events in hydra-notify plugins, and breaking the eval_* notify API: https://github.com/NixOS/hydra/pull/1145 | 23:24:49 |
| @grahamc:nixos.org | I'll probably merge tomorrow morning | 23:25:00 |
| 8 Feb 2022 |
| @grahamc:nixos.org | sigh, for my next trick I think i have to refactor GitHubStatus's 100 line long "do all the things" sub common | 00:55:36 |
| @grahamc:nixos.org | one day I'm going to go rip out alternative inputs, guh | 02:01:23 |
| @grahamc:nixos.org | Heh. https://github.com/NixOS/hydra/pull/1147 | 04:19:39 |
| @grahamc:nixos.org | Heh. https://github.com/NixOS/hydra/pull/1146 | 04:20:00 |
| cole-h | teleports behind Hydra's alt inputs and GitHubStatus plugin "Nothing personnel, kid." | 04:57:36 |
