Fix test script

[jcbhmr]

from #57 where i wanted to see it go green

doesn't work for forks

likely problem: idk
likely solution: some fiddling with the github token perms in the settings tab?

basically the token needs READONLY perms to just the "contents: read" flag across forks which could be what's happening?

[spenserblack]

Screenshot is from a PR, right? IMO the solution is that this should be a dry-run for PRs, and actually push changes for pushes to main.

[jcbhmr]

Screenshot is from a PR, right? IMO the solution is that this should be a dry-run for PRs, and actually push changes for pushes to main.

you're right. i was misunderstanding the error message. i thought it failed to pull when (youre right) it failed to push

thus something is wrong with this?

      - uses: ./
        with:
          # For now, you'll need to manually specify a GitHub token until we
          # solve #2. The x: prefix is a dummy username.
          token: x:${{ secrets.GITHUB_TOKEN }}
          dry-run: ${{ github.ref == 'main' }}

[jcbhmr]

yeah confirmed: dry-run was FALSE for some reason:

https://github.com/spenserblack/actions-wiki/actions/runs/4693381558/jobs/8320289598?pr=57

[jcbhmr]

actually yeah that makes sense. pr has github.ref of jcbhmr:some-branch in:

dry-run: ${{ 'jcbhmr:some-branch' == main }}
dry-run: false

when it should be:

dry-run: ${{ github.ref != 'main' }}
dry-run: ${{ 'jcbhmr:some-branch' != 'main' }}
dry-run: true

🎉