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

file_or_dir parameter doesn't accepts multiple files with space separated #26

Closed
dhimanmohit2548 opened this issue Aug 18, 2021 · 5 comments · Fixed by #27
Closed

file_or_dir parameter doesn't accepts multiple files with space separated #26

dhimanmohit2548 opened this issue Aug 18, 2021 · 5 comments · Fixed by #27

Comments

@dhimanmohit2548
Copy link

I was trying to use the action https://github.com/marketplace/actions/get-all-changed-files for the files changed and then using ibiqlik/action-yamllint@log-output but when supplied the output of the action jitterbit/get-changed-files@v1 to the yaml lint it doesn't work.

image

@ibiqlik
Copy link
Owner

ibiqlik commented Aug 18, 2021

@dhimanmohit2548 Can confirm, the output is enclosed in singe quotes and yamllint is treating it as a single string/path. I'll take a look at this ASAP.

@dhimanmohit2548
Copy link
Author

@ibiqlik I tried with the following inputs:

file_or_dir: ".github/workflows/blank.yml env/auto/ce.yml env/auto/gateway.yml env/auto/notifier.yml"

file_or_dir: '.github/workflows/blank.yml env/auto/ce.yml env/auto/gateway.yml env/auto/notifier.yml'

file_or_dir: .github/workflows/blank.yml env/auto/ce.yml env/auto/gateway.yml env/auto/notifier.yml

file_or_dir: "'.github/workflows/blank.yml env/auto/ce.yml env/auto/gateway.yml env/auto/notifier.yml'"

for all these three inputs I received the same error

[Errno 2] No such file or directory: '.github/workflows/blank.yml env/auto/ce.yml env/auto/gateway.yml env/auto/notifier.yml'

image

@ibiqlik
Copy link
Owner

ibiqlik commented Aug 18, 2021

@dhimanmohit2548 Can you please give it a try uses: ibiqlik/action-yamllint@fix-path-input?

@dhimanmohit2548
Copy link
Author

@ibiqlik yes this one worked! Thanks!

image

@ibiqlik
Copy link
Owner

ibiqlik commented Aug 18, 2021

Awesome, just made a release. Thanks for reporting it 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants