[core] Use pretty-quick instead of custom script

[Janpot]

Trying out for mui/toolpad#823 (comment)

[mui-bot]

	Messages
📖	Netlify deploy preview: https://deploy-preview-34062--material-ui.netlify.app/

No bundle size changes

Generated by 🚫 dangerJS against 144bacf

[michaldudak]

It seems to be working well, I think we can go ahead with this one.
Since you removed the prettier.js script, you can also remove the listChangedFiles.js (unless it's used in other repositories, cc @mui/x).

[cherniavskii]

Looks good to me!
Although I don't really use prettier scripts - I have Prettier extension for VSCode installed and it runs prettier on save automatically 👍

[michaldudak]

@cherniavskii Can you verify if you use listChangedFiles.js from the monorepo somewhere in the X codebase?

[Janpot]

Yep, also using the vscode extension here. The main benefit of removing scripts/prettier.js and relying directly on the prettier CLI is that we would automatically adopt (in CI) any new file format prettier starts supporting as it updates. Or when dependent projects add prettier plugins (we added one for prisma files in Toolpad and had to update file extensions in the prettier script in core for it to work).

[cherniavskii]

@cherniavskii Can you verify if you use listChangedFiles.js from the monorepo somewhere in the X codebase?

We don't use it

[Janpot]

Let me try this out on MUI X before we merge this

[michaldudak]

Let me try this out on MUI X before we merge this

@Janpot let me know when it's ready to be merged.

[Janpot]

It can be merged if mui/mui-x#6815 can be merged