[BUG] Shouldn't delete node_modules about prettier_action HOT 8 CLOSED

This happens because someone might not have the node_modules in gitignore. Will add an option "clean_node_folder" which defaults to True, but if you need it, you can disable it from the next version onwards.

from prettier_action.

I think node_modules normally should be in .prettierignore, and if they are not, there's probably a reason for that.
That's 2 layers of protection already, and I don't really see the need for the 3rd one.
(Or you could do git checkout -- node_modules, but still I think presuming what the users want do to is not a good policy).

from prettier_action.

I agree, just don't want to change functionality if someone uses it like that. Could you please check against prettier_action@dev to verify the new option is working?

from prettier_action.

Looks good.

from prettier_action.

Had issues with this as well, took a some time to debug why my workflow was failing.

I agree, just don't want to change functionality if someone uses it like that.

Consider switching to semantic versioning for this package so its easier to introduce breaking changes like this. E.g. with release-please

from prettier_action.

@jorgenbs I thought what I was doing is semantic versioning. v4.3 introduces this change, see changelog, v4.2 was the previous version.

from prettier_action.

@burner1024 Do you consider this bug fixed?

from prettier_action.

Yes, sure.

from prettier_action.