pantheon-systems / terminus-mass-update

A Terminus plugin to apply upstream updates to multiple sites at once
MIT License
14 stars 18 forks source link

Feature Request: Allow script to continue when merge conflict (or other error) discovered #4

Open SteveRyan-ASU opened 7 years ago

SteveRyan-ASU commented 7 years ago

When running the following command, the mass-update plugin executes as intended, and works well. terminus site:list --format=list | terminus site:mass-update:apply

But upon attempting to apply an update where the site has a merge conflict present, the script terminates upon receiving that error.

2017-02-03_09-41-15

Potential solutions

Thanks for all that you do! :-)

nstielau commented 7 years ago

Good ideas, @SteveRyan-ASU. Testing for merge conflicts conflicts doesn't seem as useful to me as something like a --continue-on-failure flag, and possible spitting out which sites errored at the end.

SteveRyan-ASU commented 7 years ago

Agreed about the approach as well. That would also allow the script to continue if it encountered other errors along the way as well. Example: trying to mass-update update a site that was still in SFTP mode would probably also result in the script stopping execution. (Untested.)