You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
release shouldn't be pushing anything. It's dangerous and could potentially break users working on a team. Let the user push things to the remote - you can even verify that the remote has everything up to date using Git; release should be doing that, and nothing more.
The above point requires Support specifying remote #135 - it's vital for any sort of git functionality to work without being overly opinionated.
If release fails to git push (as it is - doing the above points would make this one moot), the tag is still created, which means release is going to act weird if you try to call release again. It completely breaks the workflow and prevents me from using release.
In the event the problem in An argument against the <type> CLI parameter #137 arises, it's too late - the tag has already been created and pushed, and the user now has to revert (ugly) or force-push to master (dangerous) and git push origin :tag (delete the remote tag) - all of which is against all Git best practices. This makes releasedangerous if you haven't gone through all of the changes beforehand, which kind of defeats the purpose of this utility.
The text was updated successfully, but these errors were encountered:
A few things:
release
shouldn't be pushing anything. It's dangerous and could potentially break users working on a team. Let the user push things to the remote - you can even verify that the remote has everything up to date using Git;release
should be doing that, and nothing more.git
functionality to work without being overly opinionated.release
fails togit push
(as it is - doing the above points would make this one moot), the tag is still created, which meansrelease
is going to act weird if you try to call release again. It completely breaks the workflow and prevents me from usingrelease
.<type>
CLI parameter #137 arises, it's too late - the tag has already been created and pushed, and the user now has to revert (ugly) or force-push to master (dangerous) andgit push origin :tag
(delete the remote tag) - all of which is against all Git best practices. This makesrelease
dangerous if you haven't gone through all of the changes beforehand, which kind of defeats the purpose of this utility.The text was updated successfully, but these errors were encountered: