Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes #3574: Git tag needs author info, should stop deploy. #3586

Merged
merged 3 commits into from
Apr 26, 2019

Conversation

danepowell
Copy link
Contributor

Fixes #3574

Changes proposed

  • Use provided or dummy author info when creating Git tags, same as we do for commits
  • Stop deployment on failure

Steps to replicate the issue

See #3574

@danepowell danepowell added 10.x Bug Something isn't working labels Apr 25, 2019
@danepowell
Copy link
Contributor Author

I'm going to go ahead and merge this so I can get it into a patch release, but let me know if you have any feedback and we can clean it up afterwards.

@danepowell danepowell merged commit 1a4c635 into acquia:10.x Apr 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

blt deploy fails when the git user name and email are not configured
1 participant