Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Cannot recognize file pattern when using files_yaml #1286

Closed
3 tasks done
wkt-mineral opened this issue Jun 21, 2023 · 3 comments
Closed
3 tasks done

[BUG] Cannot recognize file pattern when using files_yaml #1286

wkt-mineral opened this issue Jun 21, 2023 · 3 comments
Labels
bug Something isn't working

Comments

@wkt-mineral
Copy link

Is there an existing issue for this?

  • I have searched the existing issues

Does this issue exist in the latest version?

  • I'm using the latest release

Describe the bug?

When using files_yaml with file pattern like **.js, Unresolved alias (the anchor must be set before the alias): *.sql error appears.

To Reproduce

  - name: Get changed files
    id: changed-files-yml
    uses: tj-actions/changed-files@v36
    with:
      files_yaml: |
        sql:
          - **.sql
        all:
          - **

What OS are you seeing the problem on?

ubuntu-latest or ubuntu-20.04

Expected behavior?

sql should include all .sql changed files while all should have all changed files.

Relevant log output

Run tj-actions/changed-files@v36
changed-files
Error: Unresolved alias (the anchor must be set before the alias): *.sql

Anything else?

No response

Code of Conduct

  • I agree to follow this project's Code of Conduct
@wkt-mineral wkt-mineral added the bug Something isn't working label Jun 21, 2023
@github-actions
Copy link
Contributor

Thanks for reporting this issue, don't forget to star this project if you haven't already to help us reach a wider audience.

@wkt-mineral wkt-mineral closed this as not planned Won't fix, can't repro, duplicate, stale Jun 21, 2023
@jpenna
Copy link

jpenna commented Jul 20, 2024

@wkt-mineral What is the solution?

@jackton1
Copy link
Member

jackton1 commented Jul 20, 2024

Hi @jpenna @wkt-mineral you’ll need to use quotes.


      files_yaml: |
        sql:
          - '**.sql'
        all:
          - '**'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants