| 18 Mar 2024 |
infinisil | Philip Taron (UTC-8): I gave you write access since you've been so helpful with the reviews of it before the move already, so you can merge my PRs if you want, the first one being https://github.com/NixOS/nixpkgs-check-by-name/pull/1 | 22:46:00 |
Philip Taron (UTC-8) | 👍️ | 22:46:21 |
Philip Taron (UTC-8) | Would you be opposed to me adding a flake? I can make it integrate with the default.nix you have in PR #1. | 22:48:58 |
infinisil | Philip Taron (UTC-8): Nope :) | 23:27:59 |
infinisil | Philip Taron (UTC-8): Separate PR though? | 23:28:11 |
Philip Taron (UTC-8) | Of course. | 23:28:17 |
Philip Taron (UTC-8) | Try rebasing PR #9 | 23:28:43 |
Philip Taron (UTC-8) | Hrm. No workflows ran. | 23:29:44 |
Philip Taron (UTC-8) | There we go. | 23:30:40 |
Philip Taron (UTC-8) | My brain completely yadda-yadda-yadda'd "master" vs. "main" difference | 23:31:09 |
infinisil | :P | 23:33:27 |
Philip Taron (UTC-8) | 2min run... not great, not bad. | 23:34:05 |
infinisil | I guess a lot of it comes from it fetching the dependencies for the shell | 23:34:42 |
infinisil | Maybe we could just not build the shell in CI, probably not necessary | 23:35:03 |
Philip Taron (UTC-8) | That's why I like the detsys magic cache that puts the stuff needed to build into the GitHub action cache | 23:38:13 |
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 |