Bug fix (non-breaking change which fixes an issue)
Refactor (non-breaking change which improves the structure or operation of the implementation)
Description
Revise the return code handling to ensure errors during pushes are correctly reported. In the process, streamline return code handling overall and remove use of the EXIT trap handler which made the code more complex without adding much value.
Motivation and Context
Save failures were going undetected as a result of branch protection changes.
How Has This Been Tested?
Tricky to do in isolation - simplest to do visual review and then respond quickly to any observed issues.
Intent of Change
Description
Revise the return code handling to ensure errors during pushes are correctly reported. In the process, streamline return code handling overall and remove use of the EXIT trap handler which made the code more complex without adding much value.
Motivation and Context
Save failures were going undetected as a result of branch protection changes.
How Has This Been Tested?
Tricky to do in isolation - simplest to do visual review and then respond quickly to any observed issues.
Related Changes
Prerequisite PRs:
Dependent PRs:
Consumer Actions: