The sound distributed version control system

#66 pijul push: make "changes" argument functional

Closed on November 16, 2020
loewenheim on November 15, 2020

I think a sensible way for the changes argument to work is like this: if any of the changes can’t be identified, we return an error with all the invalid changes and do nothing else. If all of them are valid, exactly those changes that have been supplied and that the remote doesn’t have will be pushed.

loewenheim added a change on November 15, 2020
This change could not be retrieved at the moment
CSC72XDCZTERVHC7OPTAXQD4FCCDKH7ZU6YV6GWXJZRWQXADWNEAC
loewenheim added a change on November 15, 2020
This change could not be retrieved at the moment
DMR6L6NW4PIAA46EQECAAL2NSTEGQW5RVFYM7A57Z2S7LIUNAJWAC
loewenheim added a change on November 15, 2020
This change could not be retrieved at the moment
YN6WKKDYF5YAGBQPTZ7FEHVYH7U7O43P6YPQCBFE64UPWY57UDYQC
loewenheim added a change on November 15, 2020
This change could not be retrieved at the moment
APDQTDGBGDN3CDXMYDHDI5FQ7BQS2NBW4LPM6Z6DYFRGM5WRW24AC
loewenheim added a change on November 15, 2020
This change could not be retrieved at the moment
6JI3NEUH2UCLWCM54GJ3DASAH7FCRCD2BOVVIU2ZBFLHN6B3NE2QC
loewenheim added a change on November 15, 2020
This change could not be retrieved at the moment
DMR6L6NW4PIAA46EQECAAL2NSTEGQW5RVFYM7A57Z2S7LIUNAJWAC
pmeunier on November 16, 2020

Sorry, I’ve had to reboot this repository again, since some of the changes it was recording were bad. This is the result of the recent change in the change format. Would you mind recording these changes again, with pijul 1.0.0-alpha.3 (published on crates.io now).

loewenheim added a change on November 16, 2020
5HF7C67M4DZMYTCIG32XEQQ662AHQMIHTHUK7TAVSO52XLMFBZPAC
main
pmeunier on November 16, 2020

Finally done! Thanks a million for your patience and your change.

pmeunier closed this discussion on November 16, 2020
loewenheim on November 16, 2020

My pleasure. I greatly enjoy contributing to this project.

loewenheim added a change on November 16, 2020
AN7IDX26RK33ZXASXLJMD4GTFWHCTHMJ6Y5C4ROCPIH33VUT2EYQC
main
loewenheim on November 16, 2020

Sorry, I forgot a change to main.rs before. I pushed it just now.

cole-h on November 16, 2020

It appears that I can’t pull your change to my local repo because of the various This change could not be retrieved at the moment changes above (maybe?). Could you please remove them from the discussion with the button?

(Of course, I could make the change manually, but this will also make it easier for @pmeunier to apply it :P)

pmeunier on November 17, 2020

Sorry for this confusion, it’s fixed now. @lowenheim, it seems that you’re not running cargo fmt before recording, I’m adding a part in the README on how to do this.