actions / checkout

Action for checking out a repo
https://github.com/features/actions
MIT License
5.93k stars 1.76k forks source link

[error]fatal couldn't find remote ref #36

Closed rster2002 closed 5 years ago

rster2002 commented 5 years ago

I'm having issues when using actions/checkout. When it runs, I get ##[error]fatal: couldn't find remote ref refs/pull/217/merge

The line can (and full log) can be found here.

The action itself is called node-swn-build.

Do I miss some config?

Thanks in advance!

jofftiquez commented 5 years ago

Same problem here

image

name: Deploy

on: 
  pull_request:
    branches:
      - master

jobs:
  deploy_to_live:
    name: Deploy prod hosting
    runs-on: ubuntu-latest
    strategy:
      matrix:
        node: [10]
    steps: 
      - uses: actions/checkout@master
      - uses: actions/setup-node@v1
        with:
          node-version: ${{ matrix.node }}
      - name: Install yarn
        run: npm install yarn@latest -g
      - name: Install dependencies
        run: yarn
      - name: Run build
        env: 
          VUE_APP_API: ${{ secrets.VUE_APP_API }}
          VUE_APP_CMS_URL: ${{ secrets.VUE_APP_CMS_URL }}
          VUE_APP_ENV: ${{ secrets.VUE_APP_ENV }}
          VUE_APP_SIGNIN_URL: ${{ secrets.VUE_APP_SIGNIN_URL }}
        run: yarn build
      - name: Run deploy
        env: 
          FIREBASE_TOKEN: ${{ secrets.FIREBASE_TOKEN }}
        run: yarn deploy:prod
TingluoHuang commented 5 years ago

did you guys merged your PR before the action workflow start running? the error means the pull request merge ref is not there anymore.

rster2002 commented 5 years ago

Oh, uhm... maybe...

That explains it. Good to know.

Thanks for clarifying.

jofftiquez commented 5 years ago

@TingluoHuang wow. I thought pull_request means after the PR has been merged. How do I do that? I want to workflow to run after the PR has been merged to master? Thanks.

TingluoHuang commented 5 years ago

i think it might be

on: 
  push:
    branches:
    - master
jofftiquez commented 5 years ago

@TingluoHuang holy cow. Thanks man. An on_merge even would be nice tho.

Alessthana commented 4 years ago

Thanks, @TingluoHuang