Narrat
Narrat
The additional force push because I forgot to remove a temporary output
Not so sure on the less code part :D I tried at least... But yeah, tomb became a chunky shell script over the time :D
No worries. You could also say what should be dropped or changed and I adjust it :D No problem to adjust the test changes onto the test you added instead...
The general change in behaviour is implemented. Adjustment of docs and tests are next on the agenda. Although I saw the CI also failed for the weblate update? Fails the...
Okay. Hints at least at the expected location if it fails at the kdf tests. But why does it at also happen at the same stage for the weblatePR?
Tests are adjusted, so this PR should be generally ready for review
It doesn't seem to be widely known (it took me a while to find a hint when I initially wrote the changes), but zsh does support optional arguments: https://zsh.sourceforge.io/Doc/Release/Zsh-Modules.html#index-zparseopts ```...
With the current changes available in this PR, the part were the argument for `--kdf` is made optional is missing. That could explain the failing test. Will force push the...
Okay, theoretically works. Checks went through. Edit: https://github.com/dyne/tomb/actions/runs/11601340887/job/32303835038?pr=527
Why size 8 and not 4 for the real tabs? It stretches greatly... Mostly curious, as I can locally adjust to size 4 :D