Open in StackBlitz

npm i https://pkg.pr.new/prettier@17331

commit:cb89f03

✅ Deploy Preview forprettier ready!

Name	Link
🔨 Latest commit	cb89f03
🔍 Latest deploy log	https://app.netlify.com/sites/prettier/deploys/67fc5e827bfebc00087112c1
😎 Deploy Preview	https://deploy-preview-17331--prettier.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to yourNetlify site configuration.

The implementation looks good.
Can you add changelog and update docs?

The implementation looks good. Can you add changelog and update docs?

Added and updated!

I'm wondering if we should call this function first. So that we can exclude some files even the file name or extension matches. So we can fix#4547 an exclude specific file likepnpm-lock.yml#14517

To do so, we need to update it to return 3 kinds of values, egtrue(yes, supported),false(no, not supported),undefined (don't care).

Alternativly, we can add a similar functionshouldIgnore(file)

To do so, we need to update it to return 3 kinds of values, eg true(yes, supported),false(no, not supported), undefined (don't care).

This is dangerous because the plugin author is very likely to returnfalse unexpectedly.

To do so, we need to update it to return 3 kinds of values, eg true(yes, supported),false(no, not supported), undefined (don't care).

This is dangerous because the plugin author is very likely to returnfalse unexpectedly.

Good point.

Alternativly, we can add a similar function shouldIgnore(file)

Seems a better choice.