-
Notifications
You must be signed in to change notification settings - Fork 53
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
Trigger notifications (twitter, gitter, etc.) upon successful nuget task, not just on successful build #299
Labels
Milestone
Comments
gep13
added a commit
to gep13/Cake.Recipe
that referenced
this issue
Jul 9, 2020
Previously, a build had to be completely successful before notifications were sent to Twitter/Gitter, etc. However, it is possible to some publishing tasks fail, but the artifacts are still sent to their end location, i.e NuGet.org. In these cases, it would make sense for the notifications to be sent, since the packages have been released. As such, the Teardown method has been updated to send notifications, when it knows artifacts have been sent successfully.
gep13
added a commit
to gep13/Cake.Recipe
that referenced
this issue
Jul 9, 2020
Previously, a build had to be completely successful before notifications were sent to Twitter/Gitter, etc. However, it is possible to some publishing tasks fail, but the artifacts are still sent to their end location, i.e NuGet.org. In these cases, it would make sense for the notifications to be sent, since the packages have been released. As such, the Teardown method has been updated to send notifications, when it knows artifacts have been sent successfully.
AdmiringWorm
added a commit
that referenced
this issue
Jul 9, 2020
(GH-299) Change how notifications are sent
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently the notifications won't be triggered if nuget push succeeds but build later fails somewhere.
Which is a bit inconsistent.
IMO it would be better handled if notifications were triggered on nuget task success regardless of the final outcome.
The text was updated successfully, but these errors were encountered: