chore(ci): Re-enable Prettier check on pull requests

[queengooborg]

Now that #28228 has landed, which introduces a review bot for Prettier formatting, I think we're ready to re-enable Prettier formatting checks. This PR does just that.

[OnkarRuikar]

The Prettier is being run already in reviewdog.yml. Why do we want to run Prettier twice? Why do we want to report the issues twice?

[yin1999]

The Prettier is being run already in reviewdog.yml. Why do we want to run Prettier twice? Why do we want to report the issues twice?

I chatted with @queengooborg about the following idea yesterday:

Maybe we could merge this with reviewdog into one workflow, as GitHub support failure with conditions. So we could only run reviewdog to post seggestions when the style check is failed. And the failed ci could still let the maintainer know there is something need to be done before merging the PR.

[OnkarRuikar]

I chatted with @queengooborg about the following idea yesterday:

Maybe we could merge this with reviewdog into one workflow, as GitHub support failure with conditions. So we could only run reviewdog to post seggestions when the style check is failed. And the failed ci could still let the maintainer know there is something need to be done before merging the PR.

We don't have to do the linting twice even in same wokflow. 🤓
Why not straight up do the --fix step? It gives us fixes as well as failure logs. Could you look at my PR that includes all the linters with reviewdog: #28240

[github-actions]

This pull request has merge conflicts that must be resolved before it can be merged.

[queengooborg]

Closing as it has been superseded.