[feature/PI-601] Set branch ref to main if branch has been deleted on workspace delete

[jameslinnell]

No description provided.

[jameslinnell]

So because the branch gets deleted it needs to run the build in "main". If it's just a regular close of PR this will remain on the branch name.

[jaklinger]

I presume that env.BRANCH_NAME isn't empty (otherwise the step below destroy--redundant-workspaces BRANCH_NAME=origin/${{ env.BRANCH_NAME }} wouldn't work)

think that means that ${{ env.BRANCH_NAME || 'main' }} will never evaluate to main?

[jaklinger]

what happens if the branch doesn't exist, does it fail? if so could use

    if: ${{ failure() }}

to catch a failed checkout, and then checkout main instead

[jameslinnell]

So BRANCH_NAME will have a value. As it's coming from the pull request. For definite. That's how destroy--redundant-workspaces BRANCH_NAME=origin/${{ env.BRANCH_NAME }} will still work as in this script the branch name is used to determine the workspace name.

But the point is if the branch doesn't exist as a branch then it will fallback to main.

Apparently this works. But I'm more than happy to concede there is an element of doubt.
There is someone asking for it to be a feature actions/checkout#512 back in 2021 and at the bottom there is someone showing a different option.

[jameslinnell]

So because the branch gets deleted it needs to run the build in "main". If it's just a regular close of PR this will remain on the branch name.
However this is only building the environment.
On line 49 below it is still using the original branch name to destroy the workspace