Action wouldn't work in workflow triggered by pull request about action-detect-and-tag-new-version

[code-ape]

Cause of the issue is this line below which is set to something like refs/pull/52/merge on pull requests and thus fails.

action-detect-and-tag-new-version/src/main.ts

Line 17 in 624bb84

await checkout(getEnv('GITHUB_REF'));

Looks like this needs to be updated to be GITHUB_HEAD_REF if present (which is for PRs) and then fall back to GITHUB_REF.

[dfreeman]

Hi @marcinwfilewave, sorry for the long silence! I seem to have completely missed this issue being filed.

I'm not sure off the top of my head why this would fail for pull requests but not other workflow triggers.

Can you tell me more about why you want to run this action for PRs? I generally wouldn't want to allow anyone opening a pull request to my repo to be able to create tags every time they push a commit to their branch, but maybe you have a development flow in mind that I hadn't considered 🙂

[pinkasey]

I just ran into this myself -
I was trying to see If I can use this cool GH Action in my repo, so my first step was to open a branch and create a new workflow, that will only run on my branch.
the lazy way to do this is:

on: pull_request

(since no other branch has this new workflow).

But that's not really something I need -
I'm now changing this to a more appropriate triger:

on:
  push:
    branches:
      # this is my test-branch:
      - eyal_ci_if_version 

[pinkasey]

BTW,

on:
  push:
    branches:
      # this is my test-branch:
      - eyal_ci_if_version 

didn't work either, I still got the same error.

I've managed to solve it by doing this:

      - uses: actions/checkout@v2
        with:
          fetch-depth: 0
          ref: eyal_ci_if_version

(I guess this GH action assume it will only be ran on master/main. which is reasonable. posting here to help others.)
(I also upgraded actions/checkout to v2)

[code-ape]

FYI. This issue still exists and is something I hit today. Will try and post if I track down root cause.