New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
codimd: update patch and fix test #48075
Conversation
No attempt on x86_64-darwin (full log) The following builds were skipped because they don't evaluate on x86_64-darwin: codimd Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: codimd Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: codimd Partial log (click to expand)
|
@GrahamcOfBorg test codimd |
Success on x86_64-linux (full log) Attempted: tests.codimd Partial log (click to expand)
|
Failure on aarch64-linux (full log) Attempted: tests.codimd Partial log (click to expand)
|
Instead of specifying the patch which links to $PR (which breaks the package each time you update or force-push that branch) it might be better to reference the actual commit from your codimd fork. |
Since it's merged it's fine like this. |
Motivation for this change
I updated my PR, which was recently merged into master of CodiMD, so the checksum of the included patch changed.
Also the test still seems to be non-deterministic after #47179, because the first request right after startup results only sometimes in a 503. Now there is a
waitUntilSucceeds
for this request, because a consecutive request always works.Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)