list-targets: Allow passing multi-line files

[shink]

Previously only comma-separated values ​​were supported:

- uses: docker/bake-action/subaction/list-targets@v5
  with:
    files: arg.json,docker-bake.hcl

Now we can pass multi-line values, which is much better:

- uses: docker/bake-action/subaction/list-targets@v5
 with:
   files: |
     arg.json
     docker-bake.hcl

I just made a quick change, can anyone review it? Thanks!

[crazy-max]

Can you also add a new job for testing in https://github.com/docker/bake-action/blob/master/.github/workflows/ci-subaction.yml

Also missing DCO, see https://github.com/docker/bake-action/pull/247/checks?check_run_id=30087916762

[crazy-max]

I think we can just use a regexp for this:

const files = `${{ inputs.files }}` ? `${{ inputs.files }}`.split(/[\r?\n,]+/) : [];

[shink]

@crazy-max thanks so much for your review! I'll do this after my holiday.

[shink]

@crazy-max It's ready for review, please take a look and trigger the CI, thanks!

[crazy-max]

Thanks! Can you squash your commits please?

[crazy-max]

The matrix looks empty: https://github.com/docker/bake-action/actions/runs/11231844927/job/31222685804#step:4:5

[shink]

ah yes, but the other two jobs are the same. I'm doing some changes to them.

[crazy-max]

ah yes, but the other two jobs are the same. I'm doing some changes to them.

I think you could just create a group-multi-files folder with docker-bake.hcl and docker-bake-2.hcl files like:

# docker-bake.hcl
group "default" {
  targets = ["t1", "t2", "t3"]
}

target "t1" {
  target = "t1"
}

target "t2" {
  target = "t2"
}

# docker-bake-2.hcl
target "t3" {
  target = "t3"
}

[crazy-max]

LGTM thanks!

[shink]

Thanks for the quick and patient review!