Check Before script failure to abort action

[JuPrgn]

Follow #9001 to apply script failure detection to others Before events as suggested : #9001 (comment)

---

✒️ I contribute this code under The Developer Certificate of Origin.

[pmiossec]

@JuPrgn just FYI (and for subsequent PRs 😁), that's a good practice to do the PR from another branch than 'master'.

[JuPrgn]

Ok I will create new branch next time with feature name

[JuPrgn]

@pmiossec to squash my commits here I use "Force push with lease" on this branch is there another clean way to do this ?

[pmiossec]

@pmiossec to squash my commits here I use "Force push with lease" on this branch is there another clean way to do this ?

No, that's the normal way to do it.

[JuPrgn]

I successfully tested BeforePush, BeforePull, BeforeCheckout, BeforeMerge and BeforeFetch using a script with :

• exit 1 action is aborted
• exit 0 action is done

It would be great to add a setting on script settings to let user select if he want do enable/disable abort on script failure. I don't know enough C# to add it myself but this could be a usefull feature.

[gerhardol]

Only looked at the changes, seem to be what is expected

[RussKie]

Thank you