bug: PR from external user/fork fails version check action

[cjyetman]

We received a PR from an external (to our GitHub org) user, from their external fork, and the CI failed the version check. I suspect it's a permission problem. The error occurs in the "checkout working HEAD" step. Note that this run probably would have failed anyway since external users don't know about bumping the version number with every change, but that was not the cause of the error here.

::group::Fetching the repository
Fetching the repository
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --no-recurse-submodules --filter=blob:none --depth=1 origin +refs/heads/ggplot2_360*:refs/remotes/origin/ggplot2_360* +refs/tags/ggplot2_360*:refs/tags/ggplot2_360*
  The process '/usr/bin/git' failed with exit code 1
  Waiting 18 seconds before trying again
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --no-recurse-submodules --filter=blob:none --depth=1 origin +refs/heads/ggplot2_360*:refs/remotes/origin/ggplot2_360* +refs/tags/ggplot2_360*:refs/tags/ggplot2_360*
  The process '/usr/bin/git' failed with exit code 1
  Waiting 15 seconds before trying again
  /usr/bin/git -c protocol.version=2 fetch --no-tags --prune --no-recurse-submodules --filter=blob:none --depth=1 origin +refs/heads/ggplot2_360*:refs/remotes/origin/ggplot2_360* +refs/tags/ggplot2_360*:refs/tags/ggplot2_360*
  ##[debug]Unsetting HOME override
  ::remove-matcher owner=checkout-git::
  ##[debug]Removed matchers: 'checkout-git'
  Error: The process '/usr/bin/git' failed with exit code 1
  ##[debug]Node Action run completed with exit code 1
  ##[debug]Save intra-action state isPost = true
  ##[debug]Save intra-action state setSafeDirectory = true
  ##[debug]Save intra-action state repositoryPath = /home/runner/work/r2dii.plot/r2dii.plot/working
  ##[debug]Finishing: checkout working HEAD

logs_33765514052.zip

[MonikaFu]

Indeed, the PR is failing on some checks. I am not familiar with all of them. Is this a breaking issue? Can we fix it within the PR itself you think? Also. @jdhoffa can you have a look?

[jdhoffa]

This is the cause of one of the failing checks.
We cannot fix it in the PR, it would need to be fixed in the action itself.

I could look into it, but @AlexAxthelm will most certainly be able to address it faster and with less effort.
Given that the check that fails is one to bump dev version numbers on a package AND it only fails when external users open a PR, it seems like not a very high priority to look into this urgently, so I would elect to wait for @AlexAxthelm to return and poke around at it once he does.

@MonikaFu in practice, feel free to merge the PR in r2di.plot with the version check failing. The other two failures are dealt with in RMI-PACTA/r2dii.plot#586