| 18 Mar 2024 |
infinisil | Philip Taron (UTC-8): In this case I'm not sure if it'll help, since effectively everything is cached already by cache.nixos.org | 23:39:28 |
willbush | In reply to @philiptaron:matrix.org That's why I like the detsys magic cache that puts the stuff needed to build into the GitHub action cache I remember seeing that before and forgot to try it out | 23:39:57 |
Philip Taron (UTC-8) | I usually found that it's like 40% faster even if that's the case. ymmv | 23:39:59 |
infinisil | Ahh yeah I could see that | 23:40:15 |
infinisil | Feel free to try! | 23:40:22 |
Philip Taron (UTC-8) | locality is a hell of a drug | 23:40:24 |
Philip Taron (UTC-8) | will do | 23:40:27 |
Philip Taron (UTC-8) | In reply to @philiptaron:matrix.org 2min run... not great, not bad. again, not bad though | 23:40:41 |
infinisil | Yeah, could be much worse | 23:44:11 |
Philip Taron (UTC-8) | infinisil: if you have the rights to edit, change the description here to reference the new repository: https://github.com/orgs/NixOS/teams/nixpkgs-check-by-name | 23:44:20 |
infinisil | Philip Taron (UTC-8): Did, but GitHub doesn't create a clickable link 😆 | 23:45:07 |
Philip Taron (UTC-8) | Even with markdown? | 23:45:22 |
infinisil | Yup 😅 | 23:45:42 |
Philip Taron (UTC-8) | Bleh | 23:45:55 |
willbush | Was rust-analyzer every brought over? I'm not seeing it | 23:46:07 |
willbush | * Was rust-analyzer ever brought over? I'm not seeing it | 23:46:18 |
infinisil | Ahh right not yet, will do now | 23:46:54 |
infinisil | https://github.com/NixOS/nixpkgs-check-by-name/pull/10 | 23:50:21 |
infinisil | Philip Taron (UTC-8): Merge vs squash? I tend to prefer merge | 23:51:14 |
Philip Taron (UTC-8) | Me too | 23:51:19 |
Philip Taron (UTC-8) | I would do rebase if it preserved signatures but it does not. | 23:51:37 |
infinisil | I'll turn off squash/rebase-merge then :) | 23:51:58 |
Philip Taron (UTC-8) | Do it. 😈 | 23:52:11 |
infinisil | Also enabled auto-merge, and requiring 1 approval for merging :P | 23:52:48 |
infinisil | Oh and requiring status checks to merge | 23:52:59 |
Philip Taron (UTC-8) | auto-merge didn't look like it took effect | 23:53:48 |
infinisil | Philip Taron (UTC-8): It can only be enabled while CI is running | 23:54:41 |
infinisil | (so it merges automatically once CI is done) | 23:54:59 |
Philip Taron (UTC-8) | I see; since it was done, I got the boring version./ | 23:55:21 |
Philip Taron (UTC-8) | * I see; since it was done, I got the boring version. | 23:55:22 |