in_progress count check is zero breaks the actionflow

[actionsflow]

in_progress count check is zero breaks the actionflow

[theowenyoung]

Thanks report, same problem happened in my workflow, I suppose that's a github api issue, but there is also a quick fix way, I'll change the default workflow later.

[theowenyoung]

https://github.com/actionsflow/actionsflow-workflow-default/blob/18fa99c63108c1eca0157021585800a4a10ee85d/.github/workflows/actionsflow.yml#L34-L40

Hi, @zhouzhuojie , Can you try with the latest script? I have pushed it. Just changed these lines.

[zhouzhuojie]

Thanks report, same problem happened in my workflow, I suppose that's a github api issue, but there is also a quick fix way, I'll change the default workflow later.

Yes, I think that may work (sorry didn't try yet), and it definitely depends on github's pagination and its implicit ordering, and it might not be efficient to query all the runs as it may return a list of all the previous workflows before.

I fixed mine with concurrency like:

concurrency: 
  group: actionsflow

jobs:
  run:
    runs-on: ubuntu-latest
    name: Run
    steps:
      - uses: actions/checkout@v2
      - name: Run Actionsflow
        uses: actionsflow/actionsflow-action@v1
        with:
          args: "build --include ${{ github.event.inputs.include || ''}} -f ${{github.event.inputs.force=='true' && 'true' || 'false'}} --verbose ${{github.event.inputs.verbose=='true' && 'true' || 'false'}}"
          json-secrets: ${{ toJSON(secrets) }}
          json-github: ${{ toJSON(github) }}
      - name: Setup act
        uses: actionsflow/setup-act-for-actionsflow@v1
      - name: Run act
        run: act --workflows ./dist/workflows --secret-file ./dist/.secrets --eventpath ./dist/event.json --env-file ./dist/.env -P ubuntu-latest=actionsflow/act-environment:v1 -P ubuntu-18.04=actionsflow/act-environment:v1

cancel-in-progress: true is optional if we just want the new run to be pending.

[theowenyoung]

@zhouzhuojie Thanks the information! I don't know this concurrency config option before, and yes, it's a perfect way to solve concurrency problem, I'll use it as the default solution!

Thanks again!

[theowenyoung]

Done. close #28