[BUG] Prettier exits with code 1 in dry run when all files are already formatted. about prettier_action HOT 7 CLOSED

@creyD sorry for the late reply. I was checking this issue and something if off.
According to the print screens "prettier options" is using --check, so the _git_changed() should not be entering block 92-132 and should run the block 133-142.

I suspect there is something on the workflow that is making the "git status -s" return true for his particular case.
@rutajdash is your workflow doing any other work on the local checked out code that might be making the "git status -s" be returning true?

I'm using a fresh workflow but I'm also using GitHub Artifacts, so that may be adding some files. It's possible.

from prettier_action.

@fsz-codeshop This seems like its happening inside the lines 94-97 in entrypoint.sh. Could you please take a look here?

from prettier_action.

Could you please try to reproduce and look if it still happens with v4.1.1? I tried a similar process and it worked alright with the current version @rutajdash

from prettier_action.

@creyD sorry for the late reply. I was checking this issue and something if off.
According to the print screens "prettier options" is using --check, so the _git_changed() should not be entering block 92-132 and should run the block 133-142.

I suspect there is something on the workflow that is making the "git status -s" return true for his particular case.
@rutajdash is your workflow doing any other work on the local checked out code that might be making the "git status -s" be returning true?

from prettier_action.

Could you please try to reproduce and look if it still happens with v4.1.1? I tried a similar process and it worked alright with the current version @rutajdash

I'll definitely test this out and get back to you.

from prettier_action.

I had the exact same issue. Running git status -s before prettier in the workflow revealed that a previous step (code style check with php-cs-fixer) created a cache file with unexpected (at least for me) name.
After changing that filename to what was already ignored in .gitignore , git status -s reported no change and prettier finished successfully.

So, I think this issue could be closed as "user error".

from prettier_action.

@rutajdash Could you please check out @semseysandor answer and reopen if needed? Thank you @semseysandor

from prettier_action.