| 9 Jan 2022 |
 Winter (she/her) | Noted — what are they used for, then? | 01:13:40 |
 piegames | Ideally, for pinging the reponsible people. At the moment, nothing I would know of. If you edit a module you'll know whom to ping, I guess | 01:19:48 |
 hexa | In reply to @winterqt:nixos.dev
Noted — what are they used for, then? wishful thinking | 01:20:03 |
| hexa | expectation of a glorious future | 01:20:09 |
| Winter (she/her) | heh | 01:21:15 |
| Winter (she/her) | so it’s still a good idea to keep them up to date? | 01:21:27 |
| hexa | if you want to get pinged use the CODEOWNERS feature instead | 01:21:40 |
| hexa | though that only works if you have commit access | 01:21:45 |
| piegames | In reply to @winterqt:nixos.dev
so it’s still a good idea to keep them up to date? Yes | 01:21:48 |
| Winter (she/her) | In reply to @hexa:lossy.network
though that only works if you have commit access yeah, since I don’t I figured that would be the next best thing | 01:22:08 |
| Winter (she/her) | is there any reason we currently don’t take them into account for automated pinging/assigning other than we just don’t atm? | 01:22:36 |
| hexa | there is an example of how to collect meta.maintainers from modules in that ofborg issue | 01:23:02 |
| hexa | someone needs to implement the feature | 01:23:10 |
| hexa | IMO the foundation should support ofBorg with money | 01:23:26 |
| Winter (she/her) | tbh i might dedicate time to implementing it, if i can get a grasp for ofborg’s codebase | 01:24:02 |
| piegames | In reply to @winterqt:nixos.dev
yeah, since I don’t I figured that would be the next best thing You might be interested in https://github.com/piegamesde/codeowners-rs as a workaround | 01:24:13 |
| Winter (she/her) | what is it? | 01:24:45 |
| piegames | A Codeowners reimplementation without stupid limitations. Let me grab you the issue(s) | 01:25:57 |
| Winter (she/her) | In reply to @winterqt:nixos.dev
tbh i might dedicate time to implementing it, if i can get a grasp for ofborg’s codebase yeah this is not bad at all, other than the fact that a local instance apparently can’t be setup easily
i’ll take a shot at it :) | 01:28:10 |
| piegames | In reply to @piegames:matrix.org
A Codeowners reimplementation without stupid limitations. Let me grab you the issue(s) https://github.com/NixOS/nixpkgs/issues/143441 | 01:28:24 |
| piegames | In reply to @winterqt:nixos.dev
yeah this is not bad at all, other than the fact that a local instance apparently can’t be setup easily
i’ll take a shot at it :) IIRC andi- hacked something together that allows you to do this. But otherwise, it's really a "if it compiles, then deploy and pray" | 01:29:45 |
| Winter (she/her) | that’s wonderful lmao | 01:30:06 |
| Winter (she/her) | Should changes be based off of release? | 02:16:50 |
| hexa | best to check existing PRs | 02:36:00 |
| Winter (she/her) | yeah, I did and figured it was okay -- just wanted to double check thoug | 02:46:50 |
| Winter (she/her) | * yeah, I did and figured it was okay -- just wanted to double check though | 02:46:52 |
 cole-h | In reply to @winterqt:nixos.dev
Should changes be based off of release? Yes indeed :) | 05:23:31 |
| Winter (she/her) | Oh god I just noticed I forgot to squash #589, whoops. Will do that in the morning. | 05:25:02 |
| Winter (she/her) | Also i believe technically its title/commit message is wrong, since both building and rust-analyzer were broken until I applied the changes, so I don’t think it’s really Darwin specific | 05:28:44 |
| Winter (she/her) | * Also I believe technically its title/commit message is wrong, since both building and rust-analyzer were broken until I applied the changes, so I don’t think it’s really Darwin specific
Should I even bother changing it? Probably will just because I have to rebase anyways but 🤷♀️ | 05:35:41 |
