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

Docs improvement for path with multiuple paths #7

Closed
MarcinOrlowski opened this issue Jun 19, 2021 · 3 comments
Closed

Docs improvement for path with multiuple paths #7

MarcinOrlowski opened this issue Jun 19, 2021 · 3 comments

Comments

@MarcinOrlowski
Copy link

Would be handy to have examples for i.e.

path Path(s) with source code to run analysis on (required)

showing the example of defining also multiple paths

@g105b
Copy link
Member

g105b commented Jun 21, 2021

Thanks. I'll add an example to https://github.com/php-actions/example-phpstan for this purpose when I get to the code later today.

@MDSLKTR
Copy link

MDSLKTR commented Jan 22, 2022

Could you post an example how to set multiple paths?

EDIT: just use spaces i think

g105b added a commit to php-actions/example-phpstan that referenced this issue Feb 9, 2022
g105b added a commit that referenced this issue Feb 9, 2022
g105b added a commit that referenced this issue Feb 9, 2022
g105b added a commit that referenced this issue Feb 9, 2022
for #7
@g105b g105b closed this as completed in bf9abe2 Feb 9, 2022
@g105b
Copy link
Member

g105b commented Feb 9, 2022

Hi, I've just been through and tested this - the solution was to use spaces, yes - but I've added some extra code to handle directories with space names in.

See here: https://github.com/php-actions/example-phpstan/blob/master/.github/workflows/ci.yml#L15

Working action: https://github.com/php-actions/example-phpstan/runs/5124500497?check_suite_focus=true

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

No branches or pull requests

3 participants