Currently, the checkout action was configured to checkout the fork, not the target repository. This led Nx to infer the base SHA from the fork repository, which is incorrect. This PR is updating the workflow to checkout the target repo instead so that the base SHA is properly identified.
Note
A potential issue is that if the workflow checkout a detached HEAD, then the name of the branch the workfow will switch to will be new-branch. This could be an issue is Sonar uses this name for multiple open PRs. Yet, two PR may have the same branch names and so the issue may already exist.
Closes #2567
Description
Currently, the checkout action was configured to checkout the fork, not the target repository. This led Nx to infer the base SHA from the fork repository, which is incorrect. This PR is updating the workflow to checkout the target repo instead so that the base SHA is properly identified.
Note
A potential issue is that if the workflow checkout a detached HEAD, then the name of the branch the workfow will switch to will be
new-branch
. This could be an issue is Sonar uses this name for multiple open PRs. Yet, two PR may have the same branch names and so the issue may already exist.