From e5c4fb6cb885edf2fca674bf9609857be03fe9f5 Mon Sep 17 00:00:00 2001 From: Florian Klink Date: Sun, 4 May 2025 21:14:57 +0300 Subject: [PATCH] docs(snix/docs/TODO): fix heading levels These don't make sense without a parent, move these one level up. Change-Id: I492e43da1d1a429e7c46b65b0c676d5d8c54fdf6 Reviewed-on: https://cl.snix.dev/c/snix/+/30466 Tested-by: besadii Reviewed-by: Jonas Chevalier Autosubmit: Florian Klink --- snix/docs/src/TODO.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/snix/docs/src/TODO.md b/snix/docs/src/TODO.md index 56df9b75e..5cc54be14 100644 --- a/snix/docs/src/TODO.md +++ b/snix/docs/src/TODO.md @@ -10,7 +10,7 @@ Please add future ideas to the issue tracker only. Before picking something from there to work on, ask in `#snix` to make sure noone is working on this, or has some specific design in mind already. -### Correctness > Performance +## Correctness > Performance A lot of the Nix behaviour isn't well documented out, and before going too deep into performance optimizations, we need to ensure we properly grasped all hidden features. This is to avoid doing a lot of "overall architecture perf-related @@ -35,7 +35,7 @@ the resulting diff noise on resulting mismtaches). in resulting store paths. -### Performance +## Performance Even while keeping in mind some of the above caveats, there's some obvious low-langing fruits that could have a good impact on performance, with somewhat limited risk of becoming obsolete in case of behaviorial changes due to @@ -68,7 +68,7 @@ correctness: split is done, as then prefetching there would only be a matter of adding it into the one `BlobReader`. -### Error cleanup +## Error cleanup - Currently, all services use snix_castore::Error, which only has two kinds (invalid request, storage error), containing an (owned) string. This is quite primitive. We should have individual error types for BS, DS, PS.