| 27 Sep 2023 |
 infinisil | Hard to have a good definition for CI changes though. The entire repo can influence CI | 17:44:41 |
 @piegames:matrix.org | In reply to @infinisil:matrix.org
CI check runs successfully, master branch updates with stricter checks, PR gets merged, master is broken Also, this is not the problem desribed in your issue, as far as I read it | 17:45:11 |
| infinisil | But maybe one could tag commits with "if this gets merged, the CI checks for all PR's need to be re-triggered" | 17:45:31 |
| @piegames:matrix.org | In reply to @infinisil:matrix.org
Hard to have a good definition for CI changes though. The entire repo can influence CI CI is a derivation and if it changes? ^^ | 17:45:38 |
| infinisil | In reply to @piegames:matrix.org
Also, this is not the problem desribed in your issue, as far as I read it True, it's applicable in the same way though | 17:45:57 |
| infinisil | Hmm let me see if it's possible to somehow re-trigger checks | 17:47:07 |
| infinisil | Though, poor ofborg could just die from that.. | 17:47:17 |
| infinisil | Okay maybe we could be very specific with it. Only re-trigger CI for PR's that touch all-packages.nix | 17:47:59 |
| @piegames:matrix.org | How about making stricter checks soft-fail for a while at first instead? | 17:48:52 |
| infinisil | That's kind of what https://github.com/NixOS/nixpkgs/issues/256788 is about | 17:49:30 |
| @piegames:matrix.org | I must admit that I am completely lost in the issue text, after reading it twice | 17:51:02 |
| infinisil | Yeah sorry I admit it's really hard to read.. | 17:51:15 |
| infinisil | Just a wall of text 😅 | 17:51:24 |
| infinisil | For the nixcon presentation it was fun trying to come up with visual explanations of things, I'll try the same here :) | 18:08:28 |
| infinisil | piegames: https://github.com/NixOS/nixpkgs/issues/256788#issuecomment-1737965650 | 19:33:48 |
| @piegames:matrix.org | https://github.com/NixOS/nixpkgs/issues/256788#issuecomment-1737979351 yeah I think this was the actual cause for my confusion | 19:48:32 |
| infinisil | Yeah, well I'm glad I was able to explain it, it's useful specific to nixpkgs-check-by-name, but a CI retrigger is a decent fix the other general problem :) | 19:49:34 |
| infinisil | * Yeah, well I'm glad I was able to explain it, it's useful specific to nixpkgs-check-by-name, but a CI retrigger is a decent fix for the other general problem :) | 19:49:44 |
| infinisil | Oh and I guess this could also be useful if it's too expensive to re-trigger the CI check for all affected PR's | 19:52:08 |
| infinisil | (not the case for nixpkgs-check-by-name) | 19:52:22 |
| infinisil | Kind of nice how this approach allows a gradual rollout of a new CI check though. And you can kind of control the amount of the rollout based on how quickly you fix the base branch when the new check starts failing on it. | 19:58:30 |
| infinisil | * Kind of nice how this approach allows a gradual rollout of a new CI check without using any extra resources though. And you can kind of control the amount of the rollout based on how quickly you fix the base branch when the new check starts failing on it. | 19:58:46 |
| infinisil | Nice, the RFC 140 tool prevented the double addition of a package: https://github.com/NixOS/nixpkgs/pull/257565 | 21:12:28 |
| 28 Sep 2023 |
| infinisil | This is fairly small, kind of neat and self-contained, reviews appreciated: https://github.com/NixOS/nixpkgs/pull/257735 | 00:06:25 |
| 2 Oct 2023 |
|  @atra1n:matrix.org joined the room. | 01:42:33 |
| infinisil | Here's the current plan for the RFC 140 migration, I'd like to talk about that in tomorrows NAT meeting and optionally schedule a longer call if it needs to be discussed further: https://github.com/NixOS/nixpkgs/issues/258650 (cc Robert Hensing (roberth) John Ericson tomberek growpotkin ( Alex Ameen ) DavHau phaer) | 20:30:22 |
| infinisil | If anybody has items they want to talk about, please add them to the agenda in the meeting notes here: https://pad.lassul.us/uIi7xeSJTW6LJUEHulZgVQ?edit | 20:32:48 |
| infinisil | * If anybody has other items they want to talk about, please add them to the agenda in the meeting notes here: https://pad.lassul.us/uIi7xeSJTW6LJUEHulZgVQ?edit | 20:32:55 |
 tomberek | I will be on a plane. ✈️ | 20:55:37 |
 DavHau | I'm not sure if I can make it tomorrow | 22:21:15 |
