-
Notifications
You must be signed in to change notification settings - Fork 588
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
Helper for Update Build Status #2141
Conversation
@matthid in theory, merging this will merge all the others, but you may want to take them separately to ease the review :) |
Update returning of context objects to allow for stepping into process
855852e
to
ebadc51
Compare
ebadc51
to
26c11e1
Compare
Conflicts should be fixed after merging BlythMeister#5 let ctx = Target.runOrDefaultAndGetContext "Deploy" //Could also use: Target.runAndGetOptionalContext "Deploy"
Target.updateBuildStatus ctx
Target.raiseIfError ctx instead of Target.runOrDefaultAndGetContext "Deploy" //Could also use: Target.runAndGetOptionalContext "Deploy"
|> Target.updateBuildStatus
|> Target.raiseIfError |
Ok replaced by #2150 (should be merged by github after merging the other one) |
Thanks for all these merges. There is huge ironey that I don't think I need the features anymore 😂 |
Care to elaborate? |
I realised I could do everything I wanted in my build failure target... |
Description
Add ability for updating build status
This extends on:
#2138
#2139
#2140
So has the features of all 3 of the previous pull requests, but adds a little bit more :)