feat(flakes): github:DeterminateSystems/*#1016
Conversation
|
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 |
As you know I prefer not to merge PRs when CI is not green, even if it is not the "fault" of this PR that CI is not green. If you'd like your PRs to get merged faster, your help in making CI green is appreciated. CI is now green so this is good to go. |
https://github.com/orgs/DeterminateSystems/repositories?q=sort:stars+archived:false