The sound distributed version control system

#408 Inconcistent command names: `mv`, `ls` vs.`remove`

Closed on April 23, 2021
hu on April 8, 2021

Just a minor nit I have run into: All pijul commands are spelled out (like remove, etc.), but two of them are abbreviated: mv and ls. IMHO it would be nicer if all commands were spelled out – and abbreviations would work consistently for all commands.

E.g. pijul list and pijul move should be the documented standard form, but pijul mv should work as well (as it does now) – just as pijul rm (which currently is not recognized).

I am mentioning this now, since it is a small thing to change at this point in time, but doing so becomes much harder post 1.0.

pmeunier on April 8, 2021

Thanks! That’s a good suggestion indeed, and sorry about the strange behaviour of the Nest, I’ll fix it soon.

hu added a change on April 8, 2021
W5HHTRPKKM4D5CTW3CKQVPVQZUAYCL5HRC5JUDBAYRX5EZ7NWH2QC
main
hu added a change on April 8, 2021
LBVUI6AXIWB5RDRK5ECT6HQANLONXJVRHCTS54USSBXA2R4J6IWAC
main
hu on April 9, 2021

I did not find the documentation in the main pijul repository. Is there something elsewhere that I need to update to be in sync with these changes?

hu on April 9, 2021

Its in the pijul/manual repository:-) The reference section is generated from the built-in help text for all I can tell.

So updating SUMMARY.md to reference list and move instead of ls and mv should be enough. I could start a discussion there and send a patch, if that helps, but I kind of doubt it.

pmeunier on April 9, 2021

Its in the pijul/manual repository:-) The reference section is generated from the built-in help text for all I can tell.

That’s right. No need to worry about it, I’ll take care of applying your changes and updating the manual during the week-end.

Thanks for the changes!

pmeunier on April 23, 2021

Thanks for the patches, I just found the time to review it, now that the Nest is working again.

pmeunier closed this discussion on April 23, 2021