-
Notifications
You must be signed in to change notification settings - Fork 105
Open
Labels
Description
Thanks for the great tool : )
Describe the bug
- I don't think the git pull described in the github actions example in this Workaround for Detached HEAD - reg-viz/reg-suit readme is necessary.
Why
# .github/workflows/reg.yml
- uses: actions/checkout@v2
with:
fetch-depth: 0
...
- name: workaround for detached HEAD
run: |
git checkout ${GITHUB_REF#refs/heads/} || git checkout -b ${GITHUB_REF#refs/heads/} && git pullIf you specify 0 for fetch-depth in actions/checkout@v2, you can get all history.
Therefore, a git pull of the detached HEAD workaround is not necessary.
Reproduced step
Fix reg-suit/README.md
# .github/workflows/reg.yml
on: [push]
jobs:
build:
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@v2
with:
fetch-depth: 0
- name: Use Node.js v10
uses: actions/setup-node@v1
with:
node-version: "10.x"
- name: npm install, build, and test
run: |
npm i
- name: workaround for detached HEAD
run: |
git checkout ${GITHUB_REF#refs/heads/}
- name: run reg-suit
run: |
npx reg-suit runReactions are currently unavailable