feat(flakes): github:DeterminateSystems/*#1016
feat(flakes): github:DeterminateSystems/*#1016Malix-Labs wants to merge 20 commits intoNixOS:mainfrom
github:DeterminateSystems/*#1016Conversation
|
Should I split it into multiple pull requests instead? |
|
I'm fine with it being a single PR, but CI is currently failing |
|
Yes true, which is also weird as it succeeded once |
github:DeterminateSystems/*
6fc23f2 to
a909d6f
Compare
|
I want to start cleaning up some old pull request, could you either fix or close yours? |
|
By fixing do you mean rebase and reorder ? |
|
the ordering looks good to me but CI is failing: (and some more failures) |
|
Okay, will fix before the end of the weekend |
|
@raboof it's the same error in https://github.com/NixOS/nixos-search/actions/runs/21092360390/job/60665340663?pr=1015 Since #1015 is very small and has basically nothing to error about, I think both errors are unrelated to my PRs |
I now see two errors: and Both of these seem different from #1015 (which seems to be #902).
Still, your PR is blocked by the error. I cannot merge PRs that are failing. You'll have to either diagnose and fix the issue, or remove the failing flakes from the PR (perhaps comment them out with a reference to the issue). |
|
#1016 (comment) by @raboof :
The problem is that this PR is failing for extraneous reasons;
The reason this PR is failing is because of #1027 and #1144; So we have 3 options:
I think option 1 is the most valid, 2 is if we are in a rush (which we are not) and option 3 is plainly incorrect |
Please look at the failure logs yourself. The failures are in |
|
I mixed PR messages, yes I think even the DeterminateSystems flakes breaking might be caused because of issues raised in #1143 as well I will try to debug them |
|
CI is still failing, are you still planning to look into this? Otherwise I think we should close this (and similar PRs) |
|
@raboof this PR is failing because of flakes that aren't touched by this PR All the https://github.com/NixOS/nixos-search/actions/runs/24127260029/job/70394803575?pr=1016 We are back to #1016 (comment) |
On the commit right before my comment, https://github.com/NixOS/nixos-search/actions/runs/24127260029/job/70394803575, this PR was still failing on
Yeah, that's my bad, I merged #1171 without noticing that it hadn't tested the flakes. That's now reverted again with #1172 though - I'll rebase this branch (though it might be something else, #1175). |
Yep dw I've noticed It's really tiresome that checks CI is nondeterministic, I really should work on #1141 |
|
(failure is now #1175) |
|
I guess this PR can be merged, then |
https://github.com/orgs/DeterminateSystems/repositories?q=sort:stars+archived:false