radicle-cli
radicle-cli copied to clipboard
Patch improvements
While working through the patch workflow, we encounted some issues.
- [x]
rad-patch --listshould fetch refs from monorepo, not working copy, so that it can be run anywhere - [x]
rad-reviewdoesn't have an option to sync review - [x]
rad-patch --listdoesn't have option to sync, so it's out of date unless yourad sync --fetchfirst - [ ]
rad-patchpushes code before patch is confirmed - [x]
rad-patch --listdoesn't show branch names - [x]
rad-remote rmhas no output if it doesn't remove anything - [ ]
rad-untrackshould have the option to remove working copy refs/branches/remotes (onby default) - [ ]
rad-trackdoesn't tell user that it setup branch and remote - [ ]
rad-track --localshould show non-default branches too (eg. patch feature branches)
'rad-patch pushes code before patch is confirmed'
I recommend removing git-push confirmation from patch creation. Instead, either check and bail out if the code is uploaded with a flag to auto-push the code, or allow patch creation without requiring the code in radicle's local storage/monorepo.
The later option would be cool, but its harder if its even possible. Users would have more freedom to use radicle patches in their own way, with any git hosting system.
I prefer the former option for practicality. The action requires the code to be in local storage/monorepo at least ahead of any conceptual sanity checks; its a soft constraint. Therefor its somewhat implicit in patch creation.
I agree about removing push confirmation, but in the case of auto-push, we're still pushing the code regardless of whether the patch is created or not. I guess it's less weird if there's no confirmation? Hmm, I'd have to look at it again.