Come together with the global Drupal community in Rotterdam, 28 Sept – 1 Oct 2026. Sessions, contribution, connection, and Early Bird savings until 8 June.
Code looks straightforward enough. So +1 in general.
I'm wondering if the 'Force: Reset --hard before tag' checkbox is usefull git tag would as far as I know only affect committed code anyway.
Another thing we might improve is the error handling. In one test the reposerver was read-only. Where I expected an error I got an 'ok' message stating "fatal: could not read Username for 'https://git.example.com': No such device or address"
Comments
Comment #3
helmo commentedCode looks straightforward enough. So +1 in general.
I'm wondering if the 'Force: Reset --hard before tag' checkbox is usefull git tag would as far as I know only affect committed code anyway.
Another thing we might improve is the error handling. In one test the reposerver was read-only. Where I expected an error I got an 'ok' message stating "fatal: could not read Username for 'https://git.example.com': No such device or address"
Comment #5
jon pughAgreed, the code wasn't catching errors.
Last push fixed that. Both for the checkout and the push. Both fire warnings if they failed, since they only fire if optionally.
Comment #6
helmo commentednice, just tried it and now works as expected.
Comment #8
helmo commentedmerged