Allow specifying commit and branch in the same field

[mstorsjo]

Allow specifying commit and branch in the same field

[mstorsjo]

As a concrete example, my workflow might look like this:

on:
  workflow_dispatch:
    inputs:
      commit:
        description: 'Commit to extract from'
        type: string
      branch:
        description: 'Branch to extract from'
        type: string

jobs:
  do-things:
    runs-on: ubuntu-latest
    steps:
      - name: Download artifacts
        uses: dawidd6/action-download-artifact@v2
        with:
          commit: ${{inputs.commit}}
          branch: ${{inputs.branch}}
          event: push
      - uses: actions/checkout@v3
        with:
          ref: ${{ (inputs.commit != '' && inputs.commit) || inputs.branch }}
          path: repo

If there would be only one single field to pass to action-download-artifact, it would make the use with other actions such as actions/checkout more convenient.

[dawidd6]

Care to open a PR for that? It looks useful indeed.

[mstorsjo]

Care to open a PR for that? It looks useful indeed.

I'm not very familiar with modern (last 20 years) JavaScript, so while I guess I could make an attempt, I think the end result is better if someone else tries :-)

[dawidd6]

I don't think we use anything modern in this action's codebase 😄 . Well, other than some 3rd party libraries...